bug-automake
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#27781: LIBOBJS should depend on LIBOBJDIR


From: Mathieu Lirzin
Subject: bug#27781: LIBOBJS should depend on LIBOBJDIR
Date: Fri, 15 Sep 2017 10:49:43 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux)

Michael Haubenwallner <address@hidden> writes:

> On 08/27/2017 04:07 PM, Mathieu Lirzin wrote:
>> Michael Haubenwallner <address@hidden> writes:
>>> On 08/23/2017 11:24 PM, Mathieu Lirzin wrote:
>>>> Michael Haubenwallner <address@hidden> writes:
>>>>> On 08/22/2017 12:40 AM, Mathieu Lirzin wrote:
>>>>>> Michael Haubenwallner <address@hidden> 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

[Prev in Thread] Current Thread [Next in Thread]