On Wed, Jul 08, 2009 at 12:58:11PM +0200, Jim Meyering wrote:
Daniel P. Berrange wrote:
...
>> From: Jim Meyering <meyering(a)redhat.com>
>> Date: Tue, 7 Jul 2009 11:22:52 +0200
>> Subject: [PATCH 5/7] make .gnulib a submodule
>>
>> This makes it so we record (via a git submodule)
>> a snapshot of whatever version of gnulib we're using,
>> and none of gnulib sources are in the libvirt repository.
>> The result is that we have as much reproducibility as when
>> we version-controlled imported copies of the gnulib sources,
>> but without the hassle of the manual process we used when
>> syncing with upstream.
>>
>> Note that when you clone libvirt, you get only the libvirt
>> repository, but when you first run ./bootstrap, it clones
>> gnulib (at the SHA1 recorded via the submodule), creating
>> the .gnulib/ hierarchy. Then, the bootstrap script runs
>> gnulib-tool to populate gnulib/ with the files that make
>> up the selected modules.
>
>
> I think perhaps we should wire this into the autogen.sh
> script. People are used to just cloning a repository and
> running autogen.sh to get things setup, so it would cause
> unneccessary confusion to also require them to run bootstrap
> seprately.
One reason not to do this (at least not unconditionally)
is that it takes pretty long. I've just timed it at 31 seconds,
even though everything is already downloaded.
That happens to be the same duration (to the second!)
of the existing ./autogen.sh script.
I was thinking that you'd only do it if you hadn't already pulled the
gnulib sub-module, or if it was out of date.
Daniel
--
|: Red Hat, Engineering, London -o-
http://people.redhat.com/berrange/ :|
|:
http://libvirt.org -o-
http://virt-manager.org -o-
http://ovirt.org :|
|:
http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|