Re: small fix of libtool.m4
Christian <[email protected]>
| Newsgroups | gmane.comp.gnu.libtool.patches |
|---|---|
| Message-ID | <[email protected]> |
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 <[email protected]> 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 >