[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] relocatable-prog-wrapper: fix test suite
From: |
Bruno Haible |
Subject: |
Re: [PATCH] relocatable-prog-wrapper: fix test suite |
Date: |
Sun, 19 Dec 2010 12:04:21 +0100 |
User-agent: |
KMail/1.9.9 |
Hi,
Ian Beckwith wrote:
> relocatable-prog-wrapper fails the test suite.
Indeed,
$ ./gnulib-tool --create-testdir ... relocatable-prog-wrapper
fails like this:
gllib/Makefile.am:148: RELOCATABLE_VIA_WRAPPER does not appear in
AM_CONDITIONAL
The problem is that RELOCATABLE_VIA_WRAPPER is defined in module
'relocatable-prog' but is used in 'relocatable-prog-wrapper'. Recall the
distinction of these two modules:
- 'relocatable-prog-wrapper' is just the source code for the wrapper program.
- 'relocatable-prog' additionally includes the macros and rules for
applying this wrapper to existing programs.
The right fix is therefore this one:
2010-12-19 Bruno Haible <address@hidden>
relocatable-prog-wrapper: Separate from relocatable-prog.
* modules/relocatable-prog (Makefile.am): Define uninstall-hook and
uninstall-relocwrapper rule here.
* modules/relocatable-prog-wrapper (Makefile.am): ... not here.
Reported by Ian Beckwith <address@hidden>.
--- modules/relocatable-prog.orig Sun Dec 19 11:56:01 2010
+++ modules/relocatable-prog Sun Dec 19 11:51:09 2010
@@ -29,6 +29,20 @@
Makefile.am:
DEFS += -DEXEEXT=\"@address@hidden"
+if RELOCATABLE_VIA_WRAPPER
+uninstall-hook: uninstall-relocwrapper
+uninstall-relocwrapper:
+ if test $(RELOCATABLE) = yes; then \
+ case '$(EXEEXT)' in \
+ .bin*) ;; \
+ *) cd $(top_builddir) && \
+ $(MAKE) $(AM_MAKEFLAGS) EXEEXT=.bin$(EXEEXT) \
+ AM_MAKEFLAGS='$(AM_MAKEFLAGS) EXEEXT=.bin$(EXEEXT)' \
+ uninstall ;; \
+ esac; \
+ fi
+.PHONY: uninstall-relocwrapper
+endif
Include:
"relocatable.h"
--- modules/relocatable-prog-wrapper.orig Sun Dec 19 11:56:01 2010
+++ modules/relocatable-prog-wrapper Sun Dec 19 11:51:20 2010
@@ -53,20 +53,6 @@
gl_FUNC_STRERROR_SEPARATE
Makefile.am:
-if RELOCATABLE_VIA_WRAPPER
-uninstall-hook: uninstall-relocwrapper
-uninstall-relocwrapper:
- if test $(RELOCATABLE) = yes; then \
- case '$(EXEEXT)' in \
- .bin*) ;; \
- *) cd $(top_builddir) && \
- $(MAKE) $(AM_MAKEFLAGS) EXEEXT=.bin$(EXEEXT) \
- AM_MAKEFLAGS='$(AM_MAKEFLAGS) EXEEXT=.bin$(EXEEXT)' \
- uninstall ;; \
- esac; \
- fi
-.PHONY: uninstall-relocwrapper
-endif
Include: