[PATCH] setup: define CYGWINFORALL for postinstall/preremove scripts

Christopher Faylor cgf-use-the-mailinglist-please@cygwin.com
Wed Aug 25 05:47:00 GMT 2010


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



More information about the Cygwin-apps mailing list