I finally got around to testing the final version of the fix. It
works fine. Thanks to everyone for advice.
Here's the final version of the patch for review.
Thanks,
Ben.
--8<--cut here-->8--
From: Ben Pfaff
Date: Sat, 13 Nov 2010 21:05:46 -08
Hi Ben,
* Ben Pfaff wrote on Mon, Jun 14, 2010 at 06:36:17PM CEST:
> Here is another approach that I had not thought of before.
> What if we added the following to
> modules/relocatable-prog-wrapper:
> *) cd $(top_builddir) && \
> $(MAKE) $(AM_MAKEFLAGS) EXEEXT=.bin$(EXEEXT
Bruno Haible writes:
>> +uninstall-hook: uninstall-relocwrapper
>> +uninstall-relocwrapper:
>> +if RELOCATABLE_VIA_LD
>> +@:
>> +else
>> +if test $(RELOCATABLE) = yes; then \
>> +case '$(EXEEXT)' in \
>> +.bin*) ;; \
>> +*) $(MAKE) uninstall EXEEXT=.bin$(EX
Hello,
* Bruno Haible wrote on Mon, Jun 14, 2010 at 02:06:55AM CEST:
> > +uninstall-hook: uninstall-relocwrapper
> > +uninstall-relocwrapper:
> > +if RELOCATABLE_VIA_LD
> > + @:
> > +else
> > + if test $(RELOCATABLE) = yes; then \
> > + case '$(EXEEXT)' in \
> > + .bin*) ;; \
>
Hi Ben,
> +uninstall-hook: uninstall-relocwrapper
> +uninstall-relocwrapper:
> +if RELOCATABLE_VIA_LD
> + @:
> +else
> + if test $(RELOCATABLE) = yes; then \
> + case '$(EXEEXT)' in \
> + .bin*) ;; \
> + *) $(MAKE) uninstall EXEEXT=.bin$(EXEEXT) ;; \
> +
Ralf Wildenhues writes:
> * Ben Pfaff wrote on Thu, Jun 10, 2010 at 07:06:53AM CEST:
>> +if RELOCATABLE_VIA_LD
>> +@:
>> +else
>> +if test $(RELOCATABLE) = yes; then \
>> +case '$(EXEEXT)' in \
>> +.bin*) ;; \
>> +*) $(MAKE) uninstall EXEEXT=.bin$(EXEEXT) ;
Hi Ben,
* Ben Pfaff wrote on Thu, Jun 10, 2010 at 07:06:53AM CEST:
> +uninstall-hook: uninstall-relocwrapper
> +uninstall-relocwrapper:
Nice.
> +if RELOCATABLE_VIA_LD
> + @:
> +else
> + if test $(RELOCATABLE) = yes; then \
> + case '$(EXEEXT)' in \
> + .bin*) ;; \
> +
Ben Pfaff writes:
> "make distuninstallcheck" noticed a weakness in the recipes that
> relocatable-maint.texi recommends for relocatable programs: "make
> uninstall" does not remove the ".bin" programs that install-reloc
> creates on some systems.
Here's my solution written up as a proper commit
"make distuninstallcheck" noticed a weakness in the recipes that
relocatable-maint.texi recommends for relocatable programs: "make
uninstall" does not remove the ".bin" programs that install-reloc
creates on some systems.
So far, the best solution that I've come up with that is generic,
that is, t
