Michael Haubenwallner <michael.haubenwall...@ssi-schaefer.com> writes:

> On 08/27/2017 04:07 PM, Mathieu Lirzin wrote:
>> Michael Haubenwallner <michael.haubenwall...@ssi-schaefer.com> writes:
>>> On 08/23/2017 11:24 PM, Mathieu Lirzin wrote:
>>>> Michael Haubenwallner <michael.haubenwall...@ssi-schaefer.com> writes:
>>>>> On 08/22/2017 12:40 AM, Mathieu Lirzin wrote:
>>>>>> Michael Haubenwallner <michael.haubenwall...@ssi-schaefer.com> writes:
>
>>>> --8<---------------cut here---------------start------------->8---
>>>
>>>>      # If LTLIBOBJS is used, we must also clear LIBOBJS (which might
>>>>      # be created by libtool as a side-effect of creating LTLIBOBJS).
>>>>      $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//;
>>>
>>> Crucial here is that trailing { if $var =~ s/^LT// }:
>>> First, leading 'LT' eventually is dropped: $var = 'LIBOBJS';
>>> If the leading 'LT' was actually found: $clean_files{'LIBOBJS'} = 
>>> MOSTLY_CLEAN;
>> 
>> I overlooked the regexp substitution.  This code is too complex for what it
>> achieves.  I have pushed the following patch in commit
>> 5521219348c55af354878583b99c5f9d66d6d38a
>> 
>
>> -      $clean_files{"\$($var)"} = MOSTLY_CLEAN if $var =~ s/^LT//;
>> +      $clean_files{"\$(LIBOBJS)"} = MOSTLY_CLEAN if $var eq "LTLIBOBJS";
>
> One more note: IMHO, the idea was to also support "LTALLOCA" here - even
> if AC_FUNC_ALLOCA does not (yet?).
>
> /haubi/

I stand corrected.

I have reverted commit 5521219348c55af354878583b99c5f9d66d6d38a

Thanks.

-- 
Mathieu Lirzin
GPG: F2A3 8D7E EB2B 6640 5761  070D 0ADE E100 9460 4D37



Reply via email to