[patch] fix bootstrap/26999

Eric Christopher echristo@apple.com
Sun Aug 27 23:50:00 GMT 2006


Andreas Schwab wrote:
> Eric Christopher <echristo@apple.com> writes:
> 
>> 2006-08-25  Eric Christopher  <echristo@apple.com>
>>
>> 	PR bootstrap/26999
>> 	* configure.in: Only disable target directories.
>> 	* configure: Regenerate.
>>
>> Index: configure.in
>> ===================================================================
>> --- configure.in	(revision 116453)
>> +++ configure.in	(working copy)
>> @@ -1311,7 +1311,14 @@ fi
>>  for dir in $configdirs $build_configdirs $target_configdirs ; do
>>    dirname=`echo $dir | sed -e s/target-//g -e s/build-//g -e s/-/_/g`
>>    if eval test x\${enable_${dirname}} "=" xno ; then
>> +    case "${dirname}" in
>> +      boehm-gc | libada | libgomp | libgfortran | libjava | libmudflap \
>> +      | libssp | zlib)
> 
> If all you want is limit this loop to target directories, why not just
> remove $configdirs and $build_configdirs from the list of words to loop
> over?  What I'm missing here?
> 

Nothing really, I added the list since it seemed that explicitly setting 
the directories would allow people to explicitly say what they'd like to 
allow to be disabled.

Don't have an opinion either way. Probably should just get a build 
person to weigh in here with what they want. I'm happy to adjust the 
patch if they want that way and test it. Just need someone to say 
something. :)

-eric



More information about the Gcc-patches mailing list