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.

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

Run installer as regular user #68

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hborchardt
Copy link

Hi, thanks for this great project!

Running the installer as admin means that the default installation folder is the admin's appdata folder. Even when installing into a user-owned location, I can only run the WSL shortcut as Admin.

As I understand it, the required admin privileges were introduced to allow the qmk_driver_installer.exe to run.

This change runs the installer as a regular user and run qmk_driver_installer.exe using shellexec to get a UAC prompt only when it is needed.

Running the installer as admin means that the default installation
folder is the admin's appdata folder, where the user that started the
installation cannot access it. The required admin privileges were
introduced to allow the qmk_driver_installer.exe to run.

Run the installer as a regular user and run qmk_driver_installer.exe
using shellexec to get a UAC prompt only when it is needed.
@zvecr zvecr added the enhancement New feature or request label Apr 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants