"Daniel P. Berrange" <berrange(a)redhat.com> wrote:
On Wed, Dec 05, 2007 at 03:08:11PM +0100, Jim Meyering wrote:
> Here's the complete patch (but still none of the new files), rebased
> and relative to the trunk. As before, this passes "make distcheck".
> And as Daniel Veillard noted, you'd need git.
>
> However the next patch will cvs-add all of the bootstrap-imported
> files, so you won't need git unless you want to try this particular patch.
> diff --git a/Makefile.am b/Makefile.am
> index eaa204e..1f2ca52 100644
> --- a/Makefile.am
> +++ b/Makefile.am
> @@ -1,8 +1,9 @@
> ## Process this file with automake to produce Makefile.in
>
> -SUBDIRS = src qemud proxy include docs @PYTHON_SUBDIR@ tests po m4 scripts
> +SUBDIRS = gnulib/lib src qemud proxy include docs @PYTHON_SUBDIR@ \
> + tests/gnulib tests po scripts
Shouldn't the gnulib/lib SUBDIR actually just be 'gnulib' and then
a gnulib/Makefile.am have SUBDIRS = lib
Likewise, the tests/gnulib, should just be in the SUBDIRS target for
the tests/Makefile.am
That aside, I'm fine with this being added to CVS now.
Actually, I've just noticed the the python tests were running
before the gnulib ones. So I'm going to move the gnulib tests
to precede the PYTHON_SUBDIR:
SUBDIRS = gnulib/lib src qemud proxy include docs tests/gnulib \
@PYTHON_SUBDIR@ tests po scripts