Skip to content

Conversation

@JohnAZoidberg
Copy link
Member

image

Signed-off-by: Daniel Schaefer <dhs@frame.work>
@JohnAZoidberg
Copy link
Member Author

Would also be nice either to elevate permissions if not run as admin or to at least provide a nice error message.

@JohnAZoidberg JohnAZoidberg requested review from kiram9 and quinchou77 May 6, 2025 09:54
Only in release builds, because running `cargo run` from non-admin
terminal won't work with the resources.

Signed-off-by: Daniel Schaefer <dhs@frame.work>
@JohnAZoidberg
Copy link
Member Author

Would also be nice either to elevate permissions if not run as admin or to at least provide a nice error message.

Done

@JohnAZoidberg
Copy link
Member Author

Found a bug, if I run a release build from the commandline, it'll always open a window and run --versions

Should always try to run as admin in release builds.
But only autorun --versions if no arguments provided.

Tested:

- [x] Release Build with args
  - [x] Run from powershell
  - [x] Run from powershell with sudo "In new window"
  - [x] Run from powershell with sudo "With input disabled"
  - [x] Doubleclick from Explorer
    - This is impossible
- [x] Release Build without args
  - [x] Run from powershell
    - This case autoruns --versions when it shouldn't
  - [x] Run from powershell with sudo "In new window"
    - This case autoruns --versions when it shouldn't
  - [x] Run from powershell with sudo "With input disabled"
  - [x] Doubleclick from Explorer
- [x] Debug Build with args
  - [x] Run from powershell
  - [x] Run from powershell with sudo "In new window"
  - [x] Run from powershell with sudo "With input disabled"
  - [x] Doubleclick from Explorer
    - This is impossible
- [x] Debug Build without args
  - [x] Run from powershell
  - [x] Run from powershell with sudo "In new window"
    - This case autoruns --versions when it shouldn't
  - [x] Run from powershell with sudo "With input disabled"
  - [x] Doubleclick from Explorer

Signed-off-by: Daniel Schaefer <dhs@frame.work>
Signed-off-by: Daniel Schaefer <dhs@frame.work>
@JohnAZoidberg JohnAZoidberg merged commit a62ad50 into main May 7, 2025
6 checks passed
@JohnAZoidberg JohnAZoidberg deleted the double-click branch May 7, 2025 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants