On Wed, Aug 25, 2010 at 05:13:05AM +0100, Andy Koppe wrote: >On 24 August 2010 22:27, Christopher Faylor wrote: >> On Tue, Aug 24, 2010 at 09:25:29PM +0100, Andy Koppe wrote: >>>Following a suggestion by Corinna, the attached patch defines a >>>variable called CYGWINFORALL in the environment of postinstall and >>>preremove scripts. It's set to "-A" when installing for all users, and >>>to an empty string when installing for "Just me". This allows it to be >>>used both directly as an argument to cygpath or mkshortcut, and in >>>conditionals using 'test -z' or 'test -n'. >>> >>>I've checked that it does indeed get defined appropriately for both >>>install types and both postinstall and preremove script. >>> >>>Changelog: >>> ?? ?? ?? * script.cc (init_run_script): Define CYGWINFORALL variable >>> ?? ?? ?? signifying install for all users or just the current user. >> >> Shouldn't the environment variable be unset entirely when not installing >> for everyone? > >You're right, it might as well.
Looks good with that change. cgf