You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by Jim Meyering <ji...@meyering.net> on 2006/12/04 19:52:27 UTC
cpp/gen/Makefile.am: guard generated source lists
Hi Alan,
This change makes "make distcheck" pass.
"make distcheck" is a standard pre-distribution sanity test.
Part of what it does is to ensure you can build from a tarball
created by "make dist". That was failing because when unpacking
the stand-alone tarball in some other position in the hierarchy,
file names like ../../gentools/... would not resolve.
With this change, they're automatically commented out in that case.
2006-12-04 Jim Meyering <ji...@meyering.net>
Changes to make "make distcheck" pass.
* gen/Makefile.am (DISTCLEANFILES): List gen-src.mk here,
rather than distributing it.
(gen-src.mk): Guard emitted java_sources and cxx_templates
templates with "if BUILD_IN_MESSAGE_TREE" conditional.
Index: gen/Makefile.am
===================================================================
--- gen/Makefile.am (revision 482266)
+++ gen/Makefile.am (working copy)
@@ -26,19 +26,20 @@
EXTRA_DIST += timestamp
$(generated_sources): timestamp
-EXTRA_DIST += gen-src.mk
+DISTCLEANFILES = gen-src.mk
gen-src.mk: timestamp
( echo 'generated_sources = \' \
&& ls *.cpp *.h | sort -u | sed 's/.*/ & \\/;$$s/ \\//' \
) > $@-t
if BUILD_IN_MESSAGE_TREE
- ( echo 'java_sources = \' \
+ ( echo if BUILD_IN_MESSAGE_TREE; \
+ echo 'java_sources = \' \
&& find $(gentools_srcdir) -name '*.java' \
- | sort -u | sed 's/.*/ & \\/;$$s/ \\//' \
- ) >> $@-t
- ( echo 'cxx_templates = \' \
+ | sort -u | sed 's/.*/ & \\/;$$s/ \\//'; \
+ echo 'cxx_templates = \' \
&& find $(gentools_dir)/templ.cpp -name '*.tmpl' \
- | sort -u | sed 's/.*/ & \\/;$$s/ \\//' \
+ | sort -u | sed 's/.*/ & \\/;$$s/ \\//'; \
+ echo endif \
) >> $@-t
endif
mv $@-t $@