close 13472 close 16662 stop On 2014-02-14 17:43, Nick Bowler wrote: > On 2/13/14, Peter Rosin wrote: >> On 2014-02-13 18:57, Peter Rosin wrote: >>> Ok, everybody seems happy. But not me, not 100% anyway. This introduces >>> an extra fork, and AFAICT, the "extra protection" is only needed when >>> func_lalib_p is called from func_ltwrapper_script_p. Should we perhaps >>> have a separate implementation in func_ltwrapper_script_p instead of >>> simply calling func_lalib_p? >>> >>> Maybe we could also lose the "sed -e 4q" part when dd limits the size? >> >> *snip* outline, real patch attached instead... >> >> Now, which patch should I push? > > The new looks a bit nicer to me although I have not tested it. > > One comment I have that applies to both patches... you should probably > be using AC_CHECK_PROGS instead of AC_PATH_PROG because the latter > requires an absolute filename. > > Specifically, AC_PATH_PROG will ignore any user-set value for DD if > it's not absolute. Ok, I'm reviving this old thing once again. There were two minor complaints with the last patch, this patch addresses both. I.e. 1. DD=dd now works to override the dd check, you don't need to specify a full path. 2. The code duplication between func_lalib_p and func_ltwrapper_script_p has been factored out. However, to address the first point, I made use of the AC_PATH_PROGS_FEATURE_CHECK macro, which was added in Autoconf 2.62, which Libtool didn't require before. But 2.62 is 6 years old by now. If anyone has problems with that, we will have to fix it later as I'm now pushing this thing. It's been brewing for far too long already... Final version attached. Cheers, Peter