Code review comment for ~jslarraz/review-tools:add-intall-deps-target

Revision history for this message
Jorge Sancho Larraz (jslarraz) wrote :

Using pkexec would be nice, but unless I'm not aware of some option, it will prompt the user for every package that needs to be installed. It could be avoided by wrapping the loop in a bash script but I think it is an overkill considering the purpose of this target.

If there is a way to "cache" user password via pkexec I will be happy to take a look at it.

Otherwise "sudo" should be ok here

« Back to merge proposal