[Libvir] 4/8 Enable the <config.h>-requiring test; fix violations
Jim Meyering
jim at meyering.net
Fri Jan 25 22:18:46 UTC 2008
Enable the <config.h>-requiring test; fix violations
Use <config.h>, not "config.h", per autoconf documentation.
* Makefile.cfg (local-checks-to-skip) [sc_require_config_h]: Enable.
* .x-sc_require_config_h: New file, to list exempted files.
* Makefile.am (EXTRA_DIST): Add .x-sc_require_config_h.
Signed-off-by: Jim Meyering <meyering at redhat.com>
---
.x-sc_require_config_h | 5 +++++
ChangeLog | 8 ++++----
Makefile.am | 3 ++-
Makefile.cfg | 1 -
proxy/libvirt_proxy.c | 2 +-
python/libvir.c | 2 +-
python/types.c | 2 +-
qemud/event.c | 2 +-
qemud/qemud.c | 2 +-
src/bridge.h | 2 +-
src/buf.c | 2 +-
src/conf.c | 2 +-
src/console.c | 2 +-
src/event.c | 2 +-
src/gnutls_1_0_compat.h | 2 +-
src/hash.c | 2 +-
src/iptables.c | 2 +-
src/libvirt.c | 2 +-
src/nodeinfo.c | 2 +-
src/openvz_conf.c | 2 +-
src/proxy_internal.c | 2 +-
src/qemu_conf.c | 2 +-
src/qemu_conf.h | 2 +-
src/qemu_driver.c | 2 +-
src/qemu_driver.h | 2 +-
src/remote_internal.c | 2 +-
src/sexpr.c | 2 +-
src/stats_linux.c | 2 +-
src/test.c | 2 +-
src/util.c | 2 +-
src/uuid.c | 2 +-
src/virsh.c | 2 +-
src/virterror.c | 2 +-
src/xen_internal.c | 2 +-
src/xen_unified.c | 2 +-
src/xend_internal.c | 2 +-
src/xm_internal.c | 2 +-
src/xml.c | 2 +-
src/xmlrpc.c | 2 +-
src/xs_internal.c | 2 +-
tests/conftest.c | 2 +-
tests/nodeinfotest.c | 2 +-
tests/qemuxml2argvtest.c | 2 +-
tests/qemuxml2xmltest.c | 2 +-
tests/reconnect.c | 2 +-
tests/sexpr2xmltest.c | 2 +-
tests/testutils.c | 2 +-
tests/virshtest.c | 2 +-
tests/xencapstest.c | 2 +-
tests/xmconfigtest.c | 2 +-
tests/xml2sexprtest.c | 2 +-
tests/xmlrpctest.c | 2 +-
52 files changed, 59 insertions(+), 54 deletions(-)
create mode 100644 .x-sc_require_config_h
diff --git a/.x-sc_require_config_h b/.x-sc_require_config_h
new file mode 100644
index 0000000..e0903ac
--- /dev/null
+++ b/.x-sc_require_config_h
@@ -0,0 +1,5 @@
+^docs/examples/info1\.c$
+^docs/examples/suspend\.c$
+^gnulib/lib/dummy\.c$
+^gnulib/tests/dummy\.c$
+^qemud/remote_protocol\.c$
diff --git a/ChangeLog b/ChangeLog
index 5493e51..7d5e058 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -555,7 +555,7 @@ Fri Dec 7 14:27:00 UTC 2007 Richard W.M. Jones <rjones at redhat.com>
Fri Dec 7 11:06:58 CET 2007 Jim Meyering <meyering at redhat.com>
- Include "config.h" in remaining non-generated files.
+ Include <config.h> in remaining non-generated files.
* proxy/libvirt_proxy.c: Likewise.
* python/libvir.c: Likewise.
* python/types.c: Likewise.
@@ -619,7 +619,7 @@ Wed Dec 5 23:57:53 CET 2007 Jim Meyering <meyering at redhat.com>
Wed Dec 5 22:38:18 CET 2007 Jim Meyering <meyering at redhat.com>
- Include "config.h".
+ Include <config.h>.
* qemud/event.c: Likewise.
* src/buf.c: Likewise.
* src/hash.c: Likewise.
@@ -673,10 +673,10 @@ Wed Dec 5 22:30:03 CET 2007 Jim Meyering <meyering at redhat.com>
* tests/nodeinfodata/linux-nodeinfo-5.txt:
* tests/nodeinfodata/linux-nodeinfo-6.txt:
* src/test.c [WITH_TEST]: Remove definition of _GNU_SOURCE that
- would conflict with the one now in "config.h".
+ would conflict with the one now in <config.h>.
* autogen.sh: Add -I gnulib/m4.
* src/conf.c, src/sexpr.c: Don't define _GNU_SOURCE.
- Instead, include "config.h".
+ Instead, include <config.h>.
* qemud/qemud.c: Remove definition of _GNU_SOURCE.
* src/openvz_driver.c: Likewise.
* src/qemu_driver.c: Likewise.
diff --git a/Makefile.am b/Makefile.am
index 5154203..e32033a 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -9,7 +9,8 @@ EXTRA_DIST = \
libvirt.spec libvirt.spec.in \
libvirt.pc libvirt.pc.in \
$(man_MANS) autobuild.sh \
- .x-sc_avoid_if_before_free
+ .x-sc_avoid_if_before_free \
+ .x-sc_require_config_h
man_MANS = virsh.1
diff --git a/Makefile.cfg b/Makefile.cfg
index ba59f29..2c1c9e7 100644
--- a/Makefile.cfg
+++ b/Makefile.cfg
@@ -49,7 +49,6 @@ local-checks-to-skip = \
sc_prohibit_quote_without_use \
sc_prohibit_quotearg_without_use \
sc_prohibit_strcmp \
- sc_require_config_h \
sc_root_tests \
sc_space_tab \
sc_sun_os_names \
diff --git a/proxy/libvirt_proxy.c b/proxy/libvirt_proxy.c
index 5ccf855..83cf39d 100644
--- a/proxy/libvirt_proxy.c
+++ b/proxy/libvirt_proxy.c
@@ -9,7 +9,7 @@
* Daniel Veillard <veillard at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#ifdef WITH_XEN
diff --git a/python/libvir.c b/python/libvir.c
index 0fe11f2..e2f7c5b 100644
--- a/python/libvir.c
+++ b/python/libvir.c
@@ -9,7 +9,7 @@
* Daniel Veillard <veillard at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include <Python.h>
#include "libvirt/libvirt.h"
diff --git a/python/types.c b/python/types.c
index 146bb24..c91d8da 100644
--- a/python/types.c
+++ b/python/types.c
@@ -7,7 +7,7 @@
* Daniel Veillard <veillard at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include "libvirt_wrap.h"
diff --git a/qemud/event.c b/qemud/event.c
index b7d75a0..ca7dc11 100644
--- a/qemud/event.c
+++ b/qemud/event.c
@@ -21,7 +21,7 @@
* Author: Daniel P. Berrange <berrange at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include <stdlib.h>
#include <string.h>
diff --git a/qemud/qemud.c b/qemud/qemud.c
index 5f3c875..77a2352 100644
--- a/qemud/qemud.c
+++ b/qemud/qemud.c
@@ -21,7 +21,7 @@
* Author: Daniel P. Berrange <berrange at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include <sys/types.h>
#include <sys/wait.h>
diff --git a/src/bridge.h b/src/bridge.h
index 52f805b..5dcca00 100644
--- a/src/bridge.h
+++ b/src/bridge.h
@@ -22,7 +22,7 @@
#ifndef __QEMUD_BRIDGE_H__
#define __QEMUD_BRIDGE_H__
-#include "config.h"
+#include <config.h>
#ifdef WITH_QEMU
diff --git a/src/buf.c b/src/buf.c
index 188d040..5bb2bd4 100644
--- a/src/buf.c
+++ b/src/buf.c
@@ -8,7 +8,7 @@
* Daniel Veillard <veillard at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include "libvirt/libvirt.h"
diff --git a/src/conf.c b/src/conf.c
index 8e66d84..8b3aae5 100644
--- a/src/conf.c
+++ b/src/conf.c
@@ -8,7 +8,7 @@
* Daniel Veillard <veillard at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include <string.h>
diff --git a/src/console.c b/src/console.c
index 6b336d9..02a9c7f 100644
--- a/src/console.c
+++ b/src/console.c
@@ -20,7 +20,7 @@
* Daniel Berrange <berrange at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#ifndef __MINGW32__
diff --git a/src/event.c b/src/event.c
index 7dcbf32..274db9d 100644
--- a/src/event.c
+++ b/src/event.c
@@ -21,7 +21,7 @@
* Author: Daniel P. Berrange <berrange at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include "event.h"
diff --git a/src/gnutls_1_0_compat.h b/src/gnutls_1_0_compat.h
index 7ec8b63..38f6954 100644
--- a/src/gnutls_1_0_compat.h
+++ b/src/gnutls_1_0_compat.h
@@ -22,7 +22,7 @@
#ifndef LIBVIRT_GNUTLS_1_0_COMPAT_H__
-#include "config.h"
+#include <config.h>
#ifdef GNUTLS_1_0_COMPAT
#define gnutls_session_t gnutls_session
diff --git a/src/hash.c b/src/hash.c
index 4e4ce60..d92c92c 100644
--- a/src/hash.c
+++ b/src/hash.c
@@ -18,7 +18,7 @@
* Daniel Veillard <veillard at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include <string.h>
#include <stdlib.h>
diff --git a/src/iptables.c b/src/iptables.c
index b59faa4..1486411 100644
--- a/src/iptables.c
+++ b/src/iptables.c
@@ -19,7 +19,7 @@
* Mark McLoughlin <markmc at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#if WITH_QEMU
diff --git a/src/libvirt.c b/src/libvirt.c
index f4ee2e0..defadc1 100644
--- a/src/libvirt.c
+++ b/src/libvirt.c
@@ -9,7 +9,7 @@
* Daniel Veillard <veillard at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include "libvirt/libvirt.h"
#include <stdio.h>
diff --git a/src/nodeinfo.c b/src/nodeinfo.c
index 1b0191b..65a2cf1 100644
--- a/src/nodeinfo.c
+++ b/src/nodeinfo.c
@@ -21,7 +21,7 @@
* Author: Daniel P. Berrange <berrange at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <string.h>
diff --git a/src/openvz_conf.c b/src/openvz_conf.c
index 4e4019c..9084184 100644
--- a/src/openvz_conf.c
+++ b/src/openvz_conf.c
@@ -27,7 +27,7 @@
#ifdef WITH_OPENVZ
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/src/proxy_internal.c b/src/proxy_internal.c
index e0e5631..a8a7962 100644
--- a/src/proxy_internal.c
+++ b/src/proxy_internal.c
@@ -10,7 +10,7 @@
#ifdef WITH_XEN
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/src/qemu_conf.c b/src/qemu_conf.c
index 0de641d..9886300 100644
--- a/src/qemu_conf.c
+++ b/src/qemu_conf.c
@@ -21,7 +21,7 @@
* Author: Daniel P. Berrange <berrange at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#ifdef WITH_QEMU
diff --git a/src/qemu_conf.h b/src/qemu_conf.h
index 7b242f7..9f09ec1 100644
--- a/src/qemu_conf.h
+++ b/src/qemu_conf.h
@@ -24,7 +24,7 @@
#ifndef __QEMUD_CONF_H
#define __QEMUD_CONF_H
-#include "config.h"
+#include <config.h>
#ifdef WITH_QEMU
diff --git a/src/qemu_driver.c b/src/qemu_driver.c
index 71a3125..f80a121 100644
--- a/src/qemu_driver.c
+++ b/src/qemu_driver.c
@@ -21,7 +21,7 @@
* Author: Daniel P. Berrange <berrange at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#ifdef WITH_QEMU
diff --git a/src/qemu_driver.h b/src/qemu_driver.h
index 66b60b7..72b397c 100644
--- a/src/qemu_driver.h
+++ b/src/qemu_driver.h
@@ -25,7 +25,7 @@
#ifndef QEMUD_DRIVER_H
#define QEMUD_DRIVER_H
-#include "config.h"
+#include <config.h>
#ifdef WITH_QEMU
diff --git a/src/remote_internal.c b/src/remote_internal.c
index 4d855ab..aa6bb5c 100644
--- a/src/remote_internal.c
+++ b/src/remote_internal.c
@@ -21,7 +21,7 @@
* Author: Richard Jones <rjones at redhat.com>
*/
-#include "config.h"
+#include <config.h>
/* Windows socket compatibility functions. */
#include "socketcompat.h"
diff --git a/src/sexpr.c b/src/sexpr.c
index bbf0791..e7ea6c8 100644
--- a/src/sexpr.c
+++ b/src/sexpr.c
@@ -10,7 +10,7 @@
* archive for more details.
*/
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/src/stats_linux.c b/src/stats_linux.c
index de73cef..d2dda88 100644
--- a/src/stats_linux.c
+++ b/src/stats_linux.c
@@ -8,7 +8,7 @@
* Richard W.M. Jones <rjones at redhat.com>
*/
-#include "config.h"
+#include <config.h>
/* This file only applies on Linux. */
#ifdef __linux__
diff --git a/src/test.c b/src/test.c
index d39207f..2afd86f 100644
--- a/src/test.c
+++ b/src/test.c
@@ -21,7 +21,7 @@
* Daniel Berrange <berrange at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#ifdef WITH_TEST
diff --git a/src/util.c b/src/util.c
index f16aaf4..0c42124 100644
--- a/src/util.c
+++ b/src/util.c
@@ -24,7 +24,7 @@
* File created Jul 18, 2007 - Shuveb Hussain <shuveb at binarykarma.com>
*/
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdarg.h>
diff --git a/src/uuid.c b/src/uuid.c
index b2022c4..ae30217 100644
--- a/src/uuid.c
+++ b/src/uuid.c
@@ -19,7 +19,7 @@
* Mark McLoughlin <markmc at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include "uuid.h"
diff --git a/src/virsh.c b/src/virsh.c
index d081008..a1b3e38 100644
--- a/src/virsh.c
+++ b/src/virsh.c
@@ -13,7 +13,7 @@
* $Id$
*/
-#include "config.h"
+#include <config.h>
#include "libvirt/libvirt.h"
#include "libvirt/virterror.h"
diff --git a/src/virterror.c b/src/virterror.c
index bf4062d..83cf7ca 100644
--- a/src/virterror.c
+++ b/src/virterror.c
@@ -8,7 +8,7 @@
* Author: Daniel Veillard <veillard at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/src/xen_internal.c b/src/xen_internal.c
index a8cecef..1292382 100644
--- a/src/xen_internal.c
+++ b/src/xen_internal.c
@@ -10,7 +10,7 @@
#ifdef WITH_XEN
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <string.h>
diff --git a/src/xen_unified.c b/src/xen_unified.c
index d46e63b..d5b2ec2 100644
--- a/src/xen_unified.c
+++ b/src/xen_unified.c
@@ -8,7 +8,7 @@
* Richard W.M. Jones <rjones at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#ifdef WITH_XEN
diff --git a/src/xend_internal.c b/src/xend_internal.c
index 61af69b..814d16f 100644
--- a/src/xend_internal.c
+++ b/src/xend_internal.c
@@ -11,7 +11,7 @@
*/
#ifdef WITH_XEN
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <sys/types.h>
diff --git a/src/xm_internal.c b/src/xm_internal.c
index 90dc1a4..57d0eb4 100644
--- a/src/xm_internal.c
+++ b/src/xm_internal.c
@@ -23,7 +23,7 @@
*/
#ifdef WITH_XEN
-#include "config.h"
+#include <config.h>
#include <dirent.h>
#include <time.h>
diff --git a/src/xml.c b/src/xml.c
index a546002..76ca91c 100644
--- a/src/xml.c
+++ b/src/xml.c
@@ -8,7 +8,7 @@
* Daniel Veillard <veillard at redhat.com>
*/
-#include "config.h"
+#include <config.h>
#include "libvirt/libvirt.h"
diff --git a/src/xmlrpc.c b/src/xmlrpc.c
index f956f0d..ac4ea6e 100644
--- a/src/xmlrpc.c
+++ b/src/xmlrpc.c
@@ -8,7 +8,7 @@
* Anthony Liguori <aliguori at us.ibm.com>
*/
-#include "config.h"
+#include <config.h>
#include "xmlrpc.h"
#include "internal.h"
diff --git a/src/xs_internal.c b/src/xs_internal.c
index 1dbbba6..6d8610c 100644
--- a/src/xs_internal.c
+++ b/src/xs_internal.c
@@ -9,7 +9,7 @@
*/
#ifdef WITH_XEN
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <string.h>
diff --git a/tests/conftest.c b/tests/conftest.c
index abb68fd..637b737 100644
--- a/tests/conftest.c
+++ b/tests/conftest.c
@@ -1,4 +1,4 @@
-#include "config.h"
+#include <config.h>
#include <unistd.h>
#include <stdlib.h>
diff --git a/tests/nodeinfotest.c b/tests/nodeinfotest.c
index 969b85c..c80e65c 100644
--- a/tests/nodeinfotest.c
+++ b/tests/nodeinfotest.c
@@ -1,4 +1,4 @@
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/tests/qemuxml2argvtest.c b/tests/qemuxml2argvtest.c
index 22e2e81..5606744 100644
--- a/tests/qemuxml2argvtest.c
+++ b/tests/qemuxml2argvtest.c
@@ -1,4 +1,4 @@
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/tests/qemuxml2xmltest.c b/tests/qemuxml2xmltest.c
index 2ab1696..1be0e46 100644
--- a/tests/qemuxml2xmltest.c
+++ b/tests/qemuxml2xmltest.c
@@ -1,4 +1,4 @@
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/tests/reconnect.c b/tests/reconnect.c
index 4c4d648..e9bdcdb 100644
--- a/tests/reconnect.c
+++ b/tests/reconnect.c
@@ -1,4 +1,4 @@
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/tests/sexpr2xmltest.c b/tests/sexpr2xmltest.c
index 6586e92..c55aa4b 100644
--- a/tests/sexpr2xmltest.c
+++ b/tests/sexpr2xmltest.c
@@ -1,4 +1,4 @@
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <string.h>
diff --git a/tests/testutils.c b/tests/testutils.c
index 4fea6b6..80b1022 100644
--- a/tests/testutils.c
+++ b/tests/testutils.c
@@ -10,7 +10,7 @@
* $Id$
*/
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/tests/virshtest.c b/tests/virshtest.c
index fd6dcbb..676cb71 100644
--- a/tests/virshtest.c
+++ b/tests/virshtest.c
@@ -1,4 +1,4 @@
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <string.h>
diff --git a/tests/xencapstest.c b/tests/xencapstest.c
index bf9ce07..1307d80 100644
--- a/tests/xencapstest.c
+++ b/tests/xencapstest.c
@@ -1,4 +1,4 @@
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/tests/xmconfigtest.c b/tests/xmconfigtest.c
index 51f4b67..424eae4 100644
--- a/tests/xmconfigtest.c
+++ b/tests/xmconfigtest.c
@@ -21,7 +21,7 @@
*
*/
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <string.h>
diff --git a/tests/xml2sexprtest.c b/tests/xml2sexprtest.c
index 2d18e60..3118379 100644
--- a/tests/xml2sexprtest.c
+++ b/tests/xml2sexprtest.c
@@ -1,5 +1,5 @@
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
diff --git a/tests/xmlrpctest.c b/tests/xmlrpctest.c
index 5fa937f..eb75f64 100644
--- a/tests/xmlrpctest.c
+++ b/tests/xmlrpctest.c
@@ -10,7 +10,7 @@
* $Id$
*/
-#include "config.h"
+#include <config.h>
#include <stdio.h>
#include <stdlib.h>
--
1.5.4.rc4.15.g215c5
More information about the libvir-list
mailing list