From 2af2dce968e3d7a7a004e7229bf49a53bf525665 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Thu, 20 Mar 2014 12:31:32 -0700 Subject: [PATCH] "make dist" did not depend on $(BUILT_SOURCES) * lib/am/distdir.am (distdir-am): New intermediate target. Interpose this target between $(distdir) and its dependent, $(DISTFILES), so that we can ensure $(BUILT_SOURCES) are all created before we begin creating $(DISTFILES). Assaf Gordon reported that "make dist" (after ./configure from a pristine clone of GNU hello) would fail due to the absence of configmake.h while compiling lib/localcharset.c. --- lib/am/distdir.am | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/lib/am/distdir.am b/lib/am/distdir.am index 3a8b0d8..40912b3 100644 --- a/lib/am/distdir.am +++ b/lib/am/distdir.am @@ -71,10 +71,13 @@ endif %?SUBDIRS% .PHONY: distdir if %?SUBDIRS% -AM_RECURSIVE_TARGETS += distdir +AM_RECURSIVE_TARGETS += distdir distdir-am endif %?SUBDIRS% -distdir: $(DISTFILES) +distdir: $(BUILT_SOURCES) + $(MAKE) $(AM_MAKEFLAGS) distdir-am + +distdir-am: $(DISTFILES) ## ## For Gnits users, this is pretty handy. Look at 15 lines ## in case some explanatory text is desirable. -- 1.9.0.258.g00eda23