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 b...@cs.stanford.edu
Date: Sat, 13
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*) ;; \
+ *)
Bruno Haible br...@clisp.org writes:
+uninstall-hook: uninstall-relocwrapper
+uninstall-relocwrapper:
+if RELOCATABLE_VIA_LD
+@:
+else
+if test $(RELOCATABLE) = yes; then \
+case '$(EXEEXT)' in \
+.bin*) ;; \
+*) $(MAKE) uninstall
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) \
Ralf Wildenhues ralf.wildenh...@gmx.de 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
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) ;; \
+
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*) ;; \
+
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, that
Ben Pfaff b...@cs.stanford.edu 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