[Libvir] gnulib: take2, trunk-relative patch

Daniel P. Berrange berrange at redhat.com
Wed Dec 5 15:56:08 UTC 2007


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.

Regards,
Dan
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 




More information about the libvir-list mailing list