search

manjaro / pamac-cli

Terminal Package Manager for Manjaro Linux with Alpm, AUR, Appstream, Flatpak and Snap support
GNU General Public License v3.0
1 stars 1 forks source link

Update cli.vala #4

Open commonsourcecs opened 2 weeks ago

commonsourcecs commented 2 weeks ago

Implement "Exit if Root".

This would close issue #3

Note: This PR makes no extra steps to address other contingencies for root as seen in lines 409, 763, etc.

guinux commented 2 weeks ago

I can't merge this because pamac is totally usable as root if AUR is disabled. I even added some code be able to build packages when used as root.

commonsourcecs commented 2 weeks ago

Then another approach or fix is needed, as users who do run pamac with sudo or as root recurrently break permissions. It also seems to cause failures in certain circumstances.

See examples here (there are many more); -Failure: https://forum.manjaro.org/t/pamac-command-line-failed-to-synchronize-aur-database/170600 -Broken Permissions: https://forum.manjaro.org/t/new-user-created-missing-permissions/169894

Would a preference be for