On 11/26/2012 08:14 PM, Crag Wolfe wrote: hi Crag, following you comment on github, most of the diffs were already in the run_as_sudo branch I've reworked my changes against the run_as_sudo branch ... but the script in there still assumes one can run sudo. I'd vote instead for *dropping the root requirement completely*. Assuming we can _always_ use rbenv (and that it would probably raise our chances of success), we can just put in the wiki a short list of packages needed for rbenv to work (which are in the number of 5 or 6) and cover all cases with a single script Also, in the run_as_sudo branch I see an additional cleanup script which we won't need when dropping the root requirement. By now I've added some 'HAVESUDO' var on top of the script which, if set to 1, runs the sudo when needed. By default such a var is set to 0 as if one install the required basic packages and provides a RBENV version, there is no need for root credentials at all. If these changes look fine to you, I'd be happy to update the README accordingly. p.s. not sure what is the right place to continue such a discussion, github or aeolus-devel? -- Giulio Fidente

On 11/27/2012 01:45 AM, Justin Clift wrote: for people interested in this, the discussion has moved to github, the patch submitted was accepted, further improved and we ended up with a script intended to be launched by a non privileged user, which uses sudo to install few dependecies and one can even disable such a behaviour and install the deps for himself still, there is an issue raised from Martin who tried to run the script as root and is facing some errors (apparently due to a bug in puppet) https://github.com/aeolus-incubator/dev-tools/issues/9 -- Giulio Fidente