Skip to content

windows: Run --versions command if double-clicked #132

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 7, 2025
Merged

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.

1 participant