[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: small fix of libtool.m4
From: |
Christian |
Subject: |
Re: small fix of libtool.m4 |
Date: |
Mon, 9 May 2016 22:24:39 -0700 |
Hey Richard,
sorry i didn’t see your patch. as i wrote in the beginning, this is one of my
first patches i committed and i might have done something wrong. In this case i
haven’t had a look at the bug list first. sorry for that.
i don’t want to say your patch is not working, since i didn’t try it yet, but
i’ll give it a try tomorrow and tell you if it fixed the problem. right now it
looks like a nice solution to me. and it does fix the problem at it’s root
source! but as is said, i’ll give it a try tomorrow and write back again.
kind regards
christian
> On May 9, 2016, at 9:55 PM, Richard PALO <address@hidden> wrote:
>
> Le 10/05/16 02:43, Christian a écrit :
>> i’ve had a look again at libtool.m4 and don’t really get why RM is set
>> wrong. obviously the _LT_CONFIG macro literally requires
>> _LT_FILEUTILS_DEFAULT, which should set RM to ‘rm -f’. I also found several
>> uses of $RM with different options, sometimes even ‘-f’. So actually i am
>> not sure what would be the best solution here.
>>
>> right now i could change the patch like proposed by Eric Blake to the one i
>> attached. It will at least fix the direct problem i encountered. But maybe
>> for the long term, someone should check and maintain the whole libtool.m4
>> file?!
>>
>> kind regards
>> christian
>
> I filed https://savannah.gnu.org/support/index.php?108987 a few months ago,
> are you saying the proposed patches don't fix this?
>
> --
> Richard PALO
>
- small fix of libtool.m4, Christian, 2016/05/06
- Re: small fix of libtool.m4, Eric Blake, 2016/05/06
- Re: small fix of libtool.m4, Christian, 2016/05/08
- Re: small fix of libtool.m4, Christian, 2016/05/09
- Re: small fix of libtool.m4, Roumen Petrov, 2016/05/09
- Re: small fix of libtool.m4, Christian, 2016/05/09
- Re: small fix of libtool.m4, Richard PALO, 2016/05/10
- Re: small fix of libtool.m4,
Christian <=
- Re: small fix of libtool.m4, Christian, 2016/05/10