4:36 p.m.
* Makefile.am (EXTRA_DIST): Distribute cfg.mk.
---
I noticed that the 0.8.7 tarball lacks cfg.mk, so running 'make
syntax-check' on that tarball is bound to have problems (without that
file, you don't exempt tests in gnulib's maint.mk that libvirt is not
interested in, and you miss out on lots of other tests that are
libvirt-specific).
Tested with 'make distcheck', then inspecting the resulting tarball.
I almost pushed this under a trivial rule, but restrained myself to
wait for a proper review. :)
Makefile.am | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 597ec61..882e676 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -51,6 +51,7 @@ EXTRA_DIST = \
autobuild.sh \
Makefile.nonreentrant \
autogen.sh \
+ cfg.mk \
examples/domain-events/events-python \
$(syntax_check_exceptions) \
$(XML_EXAMPLES)
--
1.7.3.5
12:20 p.m.
On 02/01/2011 03:36 PM, Eric Blake wrote:
* Makefile.am (EXTRA_DIST): Distribute cfg.mk.
---
I noticed that the 0.8.7 tarball lacks cfg.mk, so running 'make
syntax-check' on that tarball is bound to have problems (without that
file, you don't exempt tests in gnulib's maint.mk that libvirt is not
interested in, and you miss out on lots of other tests that are
libvirt-specific).
Tested with 'make distcheck', then inspecting the resulting tarball.
I almost pushed this under a trivial rule, but restrained myself to
wait for a proper review. :)
Makefile.am | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 597ec61..882e676 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -51,6 +51,7 @@ EXTRA_DIST = \
autobuild.sh \
Makefile.nonreentrant \
autogen.sh \
+ cfg.mk \
examples/domain-events/events-python \
$(syntax_check_exceptions) \
$(XML_EXAMPLES)
PING - this one's definitely worth including in 0.8.8.
--
Eric Blake eblake(a)redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
12:49 p.m.
On 02/01/2011 05:36 PM, Eric Blake wrote:
* Makefile.am (EXTRA_DIST): Distribute cfg.mk.
---
I noticed that the 0.8.7 tarball lacks cfg.mk, so running 'make
syntax-check' on that tarball is bound to have problems (without that
file, you don't exempt tests in gnulib's maint.mk that libvirt is not
interested in, and you miss out on lots of other tests that are
libvirt-specific).
Tested with 'make distcheck', then inspecting the resulting tarball.
I almost pushed this under a trivial rule, but restrained myself to
wait for a proper review. :)
I tried this out by removing cfg.mk and trying to run make syntax-check.
It definitely doesn't like that, so it makes sense to include the file.
ACK.
Makefile.am | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/Makefile.am b/Makefile.am
index 597ec61..882e676 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -51,6 +51,7 @@ EXTRA_DIST = \
autobuild.sh \
Makefile.nonreentrant \
autogen.sh \
+ cfg.mk \
examples/domain-events/events-python \
$(syntax_check_exceptions) \
$(XML_EXAMPLES)
1:30 p.m.
On 02/09/2011 11:49 AM, Laine Stump wrote:
On 02/01/2011 05:36 PM, Eric Blake wrote:
> * Makefile.am (EXTRA_DIST): Distribute cfg.mk.
> ---
>
> I noticed that the 0.8.7 tarball lacks cfg.mk, so running 'make
> syntax-check' on that tarball is bound to have problems (without that
> file, you don't exempt tests in gnulib's maint.mk that libvirt is not
> interested in, and you miss out on lots of other tests that are
> libvirt-specific).
>
> Tested with 'make distcheck', then inspecting the resulting tarball.
> I almost pushed this under a trivial rule, but restrained myself to
> wait for a proper review. :)
I tried this out by removing cfg.mk and trying to run make syntax-check.
It definitely doesn't like that, so it makes sense to include the file.
ACK.
Thanks; pushed.
--
Eric Blake eblake(a)redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
5035
days inactive
5043
days old
3 comments
2 participants
participants (2)
-
Eric Blake -
Laine Stump