GNU bug report logs - #27781
LIBOBJS should depend on LIBOBJDIR

Previous Next

Package: automake;

Reported by: Michael Haubenwallner <michael.haubenwallner <at> ssi-schaefer.com>

Date: Fri, 21 Jul 2017 09:09:01 UTC

Severity: normal

Tags: confirmed, fixed, patch

Done: Mathieu Lirzin <mthl <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #66 received at 27781 <at> debbugs.gnu.org (full text, mbox):

From: Mathieu Lirzin <mthl <at> gnu.org>
To: Michael Haubenwallner <michael.haubenwallner <at> ssi-schaefer.com>
Cc: 27781 <at> debbugs.gnu.org
Subject: Re: bug#27781: LIBOBJS should depend on LIBOBJDIR
Date: Fri, 15 Sep 2017 10:49:43 +0200
Michael Haubenwallner <michael.haubenwallner <at> ssi-schaefer.com> writes:

> On 08/27/2017 04:07 PM, Mathieu Lirzin wrote:
>> Michael Haubenwallner <michael.haubenwallner <at> ssi-schaefer.com> writes:
>>> On 08/23/2017 11:24 PM, Mathieu Lirzin wrote:
>>>> Michael Haubenwallner <michael.haubenwallner <at> ssi-schaefer.com> writes:
>>>>> On 08/22/2017 12:40 AM, Mathieu Lirzin wrote:
>>>>>> Michael Haubenwallner <michael.haubenwallner <at> 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




This bug report was last modified 7 years and 310 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.