[PATCHv2] build: allow 'make syntax-check' on fresh checkout

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



For good or for bad, I did a fresh checkout, ./autogen.sh, then
'configure', then 'make syntax-check', and was surprised that it
failed.  Running 'make' before 'make syntax-check' cleaned up the
issue, but this patch makes it work up front.

* cfg.mk (sc_po_check): Add prerequisite.
---

> I debated about pushing this one, but couldn't quite bring myself
> to call it a build-breaker.  However, if it gets an ACK, I see no
> negative effects to pushing before 0.9.3.

I'm glad I didn't push it yet - it didn't work for VPATH.

 cfg.mk |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/cfg.mk b/cfg.mk
index 69f35a3..65832ae 100644
--- a/cfg.mk
+++ b/cfg.mk
@@ -605,6 +605,11 @@ _autogen:
 # regenerate HACKING as part of the syntax-check
 syntax-check: $(top_srcdir)/HACKING

+# sc_po_check can fail if generated files are not built first
+sc_po_check: $(srcdir)/daemon/remote_dispatch.h
+$(srcdir)/daemon/remote_dispatch.h:
+	$(MAKE) -C daemon remote_dispatch.h
+
 # List all syntax-check exemptions:
 exclude_file_name_regexp--sc_avoid_strcase = ^tools/virsh\.c$$

-- 
1.7.4.4

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]