[pacman-dev] [PATCH 10/11] makepkg: do not ask sudo password twice

[Loui Chang]

On Thu 24 Jun 2010 18:28 +0200, Cedric Staniewski wrote:
> On 17.06.2010 17:09, Loui Chang wrote:
> > On Fri 18 Jun 2010 00:30 +1000, Allan McRae wrote:
> >> I think I have found the issue here.   We obviously have a NOPASSWD
> >> entry in our sudoers file so "sudo -l" does not require a password.
> >>
> >> So the bug is confirmed.  However the fix is not fully functional as
> >> if I have sudo installed but can not use it for pacman, then I can
> >> no longer fall back to using "su -c".  I'd choose excess password
> >> typing over functionality loss.
> > 
> > Why not just take sudo and asroot out of the equation and treat makepkg
> > as a real non-handholding executable?
> 
> I'd like to add that "sudo -l" was never meant as hand-holding. The
> intention was to support pacman-wrappers/replacements that aren't
> supposed to be run as root because they have their own logic to call
> pacman as root. The most prominent example would be yaourt, I guess.
> But since this is broken due to the 'su -c' patch, I'm fine with
> removing it again.

Yeah it just kind of bothers me that makepkg is doing all these
auxiliary functions like package installation, uninstallation, and
permissions managment. It has lost its focus.

I think those things are better placed in outside scripts (like yaourt).
It almost seems like the only thing stopping it from becoming another
yaourt is that we've dubbed the AUR as untrusted.