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

Use polkit rather than sudo #177

Open
zefr0x opened this issue Apr 25, 2023 · 3 comments
Open

Use polkit rather than sudo #177

zefr0x opened this issue Apr 25, 2023 · 3 comments

Comments

@zefr0x
Copy link

zefr0x commented Apr 25, 2023

When looking in some Linux distributions we can see that there are very few packages depending in sudo (nearly all of those optionally depending on it), while most of them especially GUI ones depends on polkit for running things as super user or another user.

In ArchLinux we have 19 packages depending in sudo while 89 depedning in polkit.

https://archlinux.org/packages/core/x86_64/sudo/

https://archlinux.org/packages/extra/x86_64/polkit/

In Gentoo we have 25 packages depending in sudo while 115 depedning in polkit.

https://packages.gentoo.org/packages/app-admin/sudo/reverse-dependencies

https://packages.gentoo.org/packages/sys-auth/polkit/reverse-dependencies


The main reasons are:

  1. We have more control on privileges using polkit than sudo. We can restrict privileges to our needs, while using sudo we will get the privileges for every thing without restrictions.
  2. Inside a desktop environment polkit is integrated to show a GUI prompt to enter your password, while sudo require a terminal to enter your password. (We will have the ability to run PINCE as a normal desktop application)

Examples of applications using polkit:


Related issues:

@zefr0x zefr0x changed the title Use polkit rather then sudo Use polkit rather than sudo Apr 25, 2023
@korcankaraokcu
Copy link
Owner

Yeah, polkit was mentioned in the past, it could be a nice addition. I'm not working actively on pince right now but I can try to implement it when I come back. Leaving the issue open for now

@authorisation
Copy link

I was having issues with gdb not being to initialize, turns out this was because I do not have sudo installed. Any reason why there is no exception printed for this in the terminal? Took me some fiddling to figure out.

@korcankaraokcu
Copy link
Owner

korcankaraokcu commented Jul 24, 2024

Any reason why there is no exception printed for this in the terminal?

All of distros we support use sudo that's why. You also don't need sudo specifically for the AppImage binary, any sudo equivalent that allows you to run a program as root will work. That "use sudo" message at the startup is just a recommendation since that's what most of the distros use

I'm not against the sudo alternatives but I just don't see it as important as the other features or improvements. I usually spend my free time on other issues. So, any PRs are welcome on this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants