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

give an option to run wayfire as root #2540

Open
rizalmart opened this issue Dec 20, 2024 · 3 comments · May be fixed by #2546
Open

give an option to run wayfire as root #2540

rizalmart opened this issue Dec 20, 2024 · 3 comments · May be fixed by #2546

Comments

@rizalmart
Copy link

rizalmart commented Dec 20, 2024

Give an option to run wayfire on root account by simply adding a command line parameter just like in Hyprland's "--i-am-not-stupid" parameter for running Hyprland on root.

@soreau
Copy link
Member

soreau commented Dec 22, 2024

Completely untested, but does this patch work? The expected usage is # wayfire --with-great-power-comes-great-responsibility.

@rizalmart
Copy link
Author

Completely untested, but does this patch work? The expected usage is # wayfire --with-great-power-comes-great-responsibility.

i wonder why this patch doesn't merged?

@soreau
Copy link
Member

soreau commented Dec 23, 2024

For one thing, it hasn't been tested. Another thing is that the reasoning behind it is questionable, since it can make the system more vulnerable to exploits. Also, you should probably not run as root unless you know why you need to. Running the display server means every process it spawns runs as root, and this is not how the system was designed to use. Finally, I think this is something easy enough to hack in a patch, if they want this behavior on their machine alone, and shouldn't be available in i.e. stock distro packages.

soreau added a commit that referenced this issue Dec 24, 2024
To run wayfire as root, pass the option --with-great-power-comes-great-responsibility.

Fixes #2540.
@soreau soreau linked a pull request Dec 24, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants