Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[Feature Request] Non-admin appreviledge? #77

Open
amwihgu opened this issue Sep 2, 2023 · 2 comments · May be fixed by #79
Open

[Feature Request] Non-admin appreviledge? #77

amwihgu opened this issue Sep 2, 2023 · 2 comments · May be fixed by #79
Labels
✨ enhancement New feature or request

Comments

@amwihgu
Copy link

amwihgu commented Sep 2, 2023

The Apps looks great!
But is requires Admin to install.
I wonder if it's fully necessary? because I use AirPods in company laptop and it's not easy to get Admin.

thank you!
@SpriteOvO
Copy link
Owner

SpriteOvO commented Sep 3, 2023 •
edited
Loading

It seems to be possible. We use CPack with NSIS to generate installer, by default, the generated installer requires admin privilege, but can probably be disabled by adding a line to CMakeLists.txt. (Reference)

set(CPACK_NSIS_DEFINES "RequestExecutionLevel user")

I'll probably check that tomorrow.

@SpriteOvO SpriteOvO added the ✨ enhancement New feature or request label Sep 3, 2023
@SpriteOvO SpriteOvO linked a pull request Sep 4, 2023 that will close this issue
Open
@amwihgu
Copy link
Author

amwihgu commented Feb 14, 2024

Hello,
Sorry for being here again.
I noticed that in Pull #79 this issue was fixed, I assumed that "install without needing admin" will be the standard in the future. However I noticed that the new release 0.4.1 beta still asks for Admin.
Will it be always like that then?

thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
✨ enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Installer requires the current user level instead of always admin level SpriteOvO/AirPodsDesktop
2 participants
@amwihgu @SpriteOvO