This is the mail archive of the cygwin-apps mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [PATCH] Two fixes for setup postinstall handling.


Christopher Faylor wrote on 10 August 2008 16:30:

> On Sun, Aug 10, 2008 at 02:21:48PM +0100, Dave Korn wrote:

>> The first hunk finds if bash is in the list of packages that have
>> scripts to run and swaps it with the first entry if so.

> If this is necessary, I don't like the idea of special casing bash.
> Can't we add a keyword to setup.ini so this could be configurable?

  Sure.  Or we could make it automagic by looking out for sh.exe when it
goes by - using the same string we know we're going to be using to invoke
the scripts later?

>>  The first hunk could be obviated by doing away with postinstall scripts
>> for bash, but it a) is perhaps an easier change than respinning a fresh
>> bash release,
> 
> ?  How is patching, reviewing the patch, rebuilding, and releasing
> setup.exe easier than Eric releasing a new version of bash which just
> installs sh.exe unconditionally?

  Depends how busy Eric is.

>> b) is defensive programming - it's always going to be the case that, if
>> there are any scripts needed to install the shell, they should be the
>> very first thing we do, and this means that should a situation ever
>> arise in the future where we find ourselves needing bash postinstall
>> scripts, they will just work and we won't regress to the situation we
>> find ourselves in now.
> 
> Sorry but I'm not yet convinced that this is necessary.

  Well, it's still "just plain wrong" to rename a script to ".done" when you
know for absolute certain that it hasn't been.


    cheers,
      DaveK
-- 
Can't think of a witty .sigline today....


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]