-
Notifications
You must be signed in to change notification settings - Fork 590
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
docs: clarify to run "sudo firecfg" as a normal (desktop) user #6677
base: master
Are you sure you want to change the base?
Conversation
@@ -206,6 +206,9 @@ $ firejail --list | |||
|
|||
## Desktop integration | |||
|
|||
Note: Desktop integration is only applied to the user running firecfg, so make | |||
sure to run `sudo firecfg` as a normal (desktop) user, not as root. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The alternatives is
# firecfg
# firecfg --add-users YOUR_USER
$ firecfg --fix
Half the story.
|
End users following the steps for desktop integration may end up running `sudo firecfg` as root (or as a similar account) rather than as a normal desktop user. In that case, programs opened through a desktop launcher would still not be running under firejail, which might surprise users. So clarify that `sudo firecfg` should be executed as a normal (desktop) user for desktop integration. Relates to netblue30#6657. Kind of relates to netblue30#5812. Reported-by: @ginto37
f45266d
to
2aed3ce
Compare
I'm not sure the phrase "normal (desktop) user" really helps here, and I'd suggest not going with that. My admin account, the one with I feel like there's a fundamental difference in views here regarding the use of accounts on Linux. Basically I feel like you're looking at this from a different angle and it's leading you to write up the instructions from a slightly unhelpful viewpoint that's going to confuse users rather than help them. For that reason, Rusty-snake's suggestion should be the default method, not the alternative, IMO, otherwise users like myself are going to wonder why Rusty-snake's method doesn't involve modifying the In essence,
OOC, where would |
I still don't get why you want to modify it.
Each user. Keep in mind that firejails primary target are single user desktop systems. |
End users following the steps for desktop integration may end up running
sudo firecfg
as root (or as a similar account) rather than as a normaldesktop user.
In that case, programs opened through a desktop launcher would still not
be running under firejail, which might surprise users.
So clarify that
sudo firecfg
should be executed as a normal (desktop)user for desktop integration.
Relates to #6657.
Kind of relates to #5812.
Reported-by: @ginto37