>>> "RW" == Ralf Wildenhues <[EMAIL PROTECTED]> writes:
[...] RW> [3] The $(MAKE) invocation for the subpackage `distdir' is not RW> portable to all `make' implementation (as not all allow to override RW> set macros from the command line); this requirement does not change RW> with the patch. It looks portable to me (as long as you don't expect your assignment to hold for the submakes of the one being launched, of course, but that's not needed here). RW> Allow package trees (packages with subpackages) to share RW> common distributed auxiliary files (and directories) that RW> reside within a subpackage. RW> * lib/am/distdir.am (distdir %?TOPDIR_P%): Do not fail if RW> `$(distdir)' already exists. RW> (%?SUBDIRS%): Set `am__remove_distdir' to `:' to prevent RW> removal of subpackage trees for distribution. RW> * tests/subpkg3.test: New test. RW> * tests/Makefile.am: Updated. "Update." :) [...] RW> --- lib/am/distdir.am 3 Oct 2005 20:48:59 -0000 1.62 RW> +++ lib/am/distdir.am 12 Jan 2006 20:21:07 -0000 RW> @@ -50,7 +50,7 @@ RW> ## RW> if %?TOPDIR_P% RW> $(am__remove_distdir) RW> - mkdir $(distdir) RW> + test -d $(distdir) || mkdir $(distdir) RW> endif %?TOPDIR_P% RW> ## RW> ## RW> @@ -175,6 +175,7 @@ RW> $(MAKE) $(AM_MAKEFLAGS) \ RW> top_distdir="$$top_distdir" \ RW> distdir="$$distdir/$$subdir" \ RW> + am__remove_distdir=: \ RW> distdir) \ Somme comment along the lines of ## Disable am__remove_distdir so that sub-package do not clear a ## directory we have already cleared and might even have populated ## populated (e.g. shared AUX dir in the sub-package.) maybe ? At least it sure would help my bad memories. Otherwise the patch looks fine to me, please install! +cat >subpkg/foobar <<'END' +foobar +END + +cat >subpkg/Makefile.am <<'END' +SUBDIRS = +END It doesn't matter to me here, but those could be echos if you care for speed (I think to remember you do). -- Alexandre Duret-Lutz Shared books are happy books. http://www.bookcrossing.com/friend/gadl
