willeccles / cpm

🌵 A wrapper for package managers to make them consistent for those of us who are lazy.
MIT License
69 stars 13 forks source link

prt-get and guix do not use $SUDO? #56

Closed willeccles closed 2 years ago

willeccles commented 2 years ago

I'm not familiar with guix, so this may be appropriate for guix, but prt-get seems like it would require root.

willeccles commented 2 years ago

@xgqt, you added support for guix. Can you comment on this?

willeccles commented 2 years ago

And @foxsouns, you added Crux support. Thoughts?

foxsouns commented 2 years ago

And @foxsouns, you added Crux support. Thoughts?

that was an oversight: prt-get should be ran with $SUDO.

xgqt commented 2 years ago

The case of Guix is similar to Nix, so: ran rootless it manages environment of the user and with root access can modify the whole system. If I would force the use of sudo them cpm could not work for the user, right now it should be able to both manage user with $ cpm ... and system with $ sudo cpm ....

willeccles commented 2 years ago

And @foxsouns, you added Crux support. Thoughts?

that was an oversight: prt-get should be ran with $SUDO.

@foxsouns Could you open a PR to add this as necessary?

willeccles commented 2 years ago

The case of Guix is similar to Nix, so: ran rootless it manages environment of the user and with root access can modify the whole system. If I would force the use of sudo them cpm could not work for the user, right now it should be able to both manage user with $ cpm ... and system with $ sudo cpm ....

Alright, that makes sense. Will leave it off for Guix.