This is the mail archive of the cygwin-apps@sourceware.cygnus.com 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]

Re: Additional Patch for bash


>> The problem that is fixed by my patch is as follows:
>>	
>> bash-distribution uses an old config.guess which creates results like:
>>	
>> i686-pc-cygwin32	
>>	
>> and then later checks for *cygwin32 in the configure script.
>>	
>> recent versions of config.guess return	
>>	
>> i686-pc-cygwin   (!! no 32 at the end !!)
>>	
>> and, as a result, configure does not react correctly and a bash build on base
>> of
>> this configure stackdumps.
>>	
>> This behaviour is reproduceable with either the original bash-2.04.tar.gz or
>> the
>> bash-2.0.4-src.tar.gz in the LATEST directory on sourceware.
>>	
>> Greetings,
>>	
>> Michael Ring
>
>But, I didn't/don't have your problem.  So, why should we do a patch.  The real
>patch is to replace the config.guess, config.sub in bash after unarchiving the
>source.
>
>Regards,
>

You are right in one point, the patch should also include config.guess and
config.sub to make sense; because if you apply the patch without exchanging
config.guess then bash will also stackdump.

Here's the important line from configure:

-*-cygwin32*)   opt_bash_malloc=no ;;   # Cygnus's CYGWIN32 environment
+*-cygwin*)     opt_bash_malloc=no ;;   # Cygnus's CYGWIN32 environment 

After exchanging config.guess opt_bash_malloc will not be set to no ==>
Stackdump during initialisation of bash.

Did you build bash inside the cygwin sourcetree or standalone ?

I built it standalone; perhaps there is a difference.

Greetings again,

Michael Ring

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