[libvirt] [libvirt-php 0/2] Another update for building system

Hi all: This patch updeted the building system. * Added necessary checkings in configure.ac * Cleared up libvirt-php.spec and src/Makefile.am BTW, I've changed my email to business address. Lyre (2): spec file cleanup Added checkings for tools and libraries aclocal.m4 | 189 ++++++++++++++++++++++++++++++++++++++++++++++++++++++ configure.ac | 12 ++++ libvirt-php.spec | 8 +- src/Makefile.am | 30 +++------ 4 files changed, 216 insertions(+), 23 deletions(-)

From: Lyre <4179e1@gmail.com> * libvirt-php.spec: removed unnecessary macro 0{%sles_version} --- libvirt-php.spec | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libvirt-php.spec b/libvirt-php.spec index 2c6292e..51a9add 100644 --- a/libvirt-php.spec +++ b/libvirt-php.spec @@ -1,6 +1,6 @@ %define req_libvirt_version 0.6.2 -%if 0%{?suse_version} || 0%{?sles_version} +%if 0%{?suse_version} %define php_confdir %{_sysconfdir}/php5/conf.d %define php_extdir %{_libdir}/php5/extensions %else @@ -13,7 +13,7 @@ Version: 0.4 Release: 1%{?dist}%{?extra_release} Summary: PHP language binding for Libvirt -%if 0%{?suse_version} || 0%{?sles_version} +%if 0%{?suse_version} Group: Development/Libraries/PHP %else Group: Development/Libraries @@ -26,13 +26,13 @@ BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root BuildRequires: php-devel BuildRequires: libvirt-devel >= %{req_libvirt_version} BuildRequires: libxml2-devel -%if 0%{?suse_version} || 0%{?sles_version} +%if 0%{?suse_version} BuildRequires: xhtml-dtd %else BuildRequires: xhtml1-dtds %endif Requires: libvirt >= %{req_libvirt_version} -%if 0%{?suse_version} || 0%{?sles_version} +%if 0%{?suse_version} Requires: php5 %else Requires: php -- 1.7.1

* configure.ac: Added checkings for compiler, sed, install, and so on; Added AM_PATH_XML2 to check libxml2. * src/Makefile.am: dropped needless macros since it is done by autoconf; Use $INSTALL to install files rather than cp. --- aclocal.m4 | 189 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ configure.ac | 12 ++++ src/Makefile.am | 30 +++------ 3 files changed, 212 insertions(+), 19 deletions(-) diff --git a/aclocal.m4 b/aclocal.m4 index a1fa316..982143e 100644 --- a/aclocal.m4 +++ b/aclocal.m4 @@ -19,6 +19,195 @@ You have another version of autoconf. It may work, but is not guaranteed to. If you have problems, you may need to regenerate the build system entirely. To do so, use the procedure documented by the package, typically `autoreconf'.])]) +# Configure paths for LIBXML2 +# Mike Hommey 2004-06-19 +# use CPPFLAGS instead of CFLAGS +# Toshio Kuratomi 2001-04-21 +# Adapted from: +# Configure paths for GLIB +# Owen Taylor 97-11-3 + +dnl AM_PATH_XML2([MINIMUM-VERSION, [ACTION-IF-FOUND [, ACTION-IF-NOT-FOUND]]]) +dnl Test for XML, and define XML_CPPFLAGS and XML_LIBS +dnl +AC_DEFUN([AM_PATH_XML2],[ +AC_ARG_WITH(xml-prefix, + [ --with-xml-prefix=PFX Prefix where libxml is installed (optional)], + xml_config_prefix="$withval", xml_config_prefix="") +AC_ARG_WITH(xml-exec-prefix, + [ --with-xml-exec-prefix=PFX Exec prefix where libxml is installed (optional)], + xml_config_exec_prefix="$withval", xml_config_exec_prefix="") +AC_ARG_ENABLE(xmltest, + [ --disable-xmltest Do not try to compile and run a test LIBXML program],, + enable_xmltest=yes) + + if test x$xml_config_exec_prefix != x ; then + xml_config_args="$xml_config_args" + if test x${XML2_CONFIG+set} != xset ; then + XML2_CONFIG=$xml_config_exec_prefix/bin/xml2-config + fi + fi + if test x$xml_config_prefix != x ; then + xml_config_args="$xml_config_args --prefix=$xml_config_prefix" + if test x${XML2_CONFIG+set} != xset ; then + XML2_CONFIG=$xml_config_prefix/bin/xml2-config + fi + fi + + AC_PATH_PROG(XML2_CONFIG, xml2-config, no) + min_xml_version=ifelse([$1], ,2.0.0,[$1]) + AC_MSG_CHECKING(for libxml - version >= $min_xml_version) + no_xml="" + if test "$XML2_CONFIG" = "no" ; then + no_xml=yes + else + XML_CPPFLAGS=`$XML2_CONFIG $xml_config_args --cflags` + XML_LIBS=`$XML2_CONFIG $xml_config_args --libs` + xml_config_major_version=`$XML2_CONFIG $xml_config_args --version | \ + sed 's/\([[0-9]]*\).\([[0-9]]*\).\([[0-9]]*\)/\1/'` + xml_config_minor_version=`$XML2_CONFIG $xml_config_args --version | \ + sed 's/\([[0-9]]*\).\([[0-9]]*\).\([[0-9]]*\)/\2/'` + xml_config_micro_version=`$XML2_CONFIG $xml_config_args --version | \ + sed 's/\([[0-9]]*\).\([[0-9]]*\).\([[0-9]]*\)/\3/'` + if test "x$enable_xmltest" = "xyes" ; then + ac_save_CPPFLAGS="$CPPFLAGS" + ac_save_LIBS="$LIBS" + CPPFLAGS="$CPPFLAGS $XML_CPPFLAGS" + LIBS="$XML_LIBS $LIBS" +dnl +dnl Now check if the installed libxml is sufficiently new. +dnl (Also sanity checks the results of xml2-config to some extent) +dnl + rm -f conf.xmltest + AC_TRY_RUN([ +#include <stdlib.h> +#include <stdio.h> +#include <string.h> +#include <libxml/xmlversion.h> + +int +main() +{ + int xml_major_version, xml_minor_version, xml_micro_version; + int major, minor, micro; + char *tmp_version; + + system("touch conf.xmltest"); + + /* Capture xml2-config output via autoconf/configure variables */ + /* HP/UX 9 (%@#!) writes to sscanf strings */ + tmp_version = (char *)strdup("$min_xml_version"); + if (sscanf(tmp_version, "%d.%d.%d", &major, &minor, µ) != 3) { + printf("%s, bad version string from xml2-config\n", "$min_xml_version"); + exit(1); + } + free(tmp_version); + + /* Capture the version information from the header files */ + tmp_version = (char *)strdup(LIBXML_DOTTED_VERSION); + if (sscanf(tmp_version, "%d.%d.%d", &xml_major_version, &xml_minor_version, &xml_micro_version) != 3) { + printf("%s, bad version string from libxml includes\n", "LIBXML_DOTTED_VERSION"); + exit(1); + } + free(tmp_version); + + /* Compare xml2-config output to the libxml headers */ + if ((xml_major_version != $xml_config_major_version) || + (xml_minor_version != $xml_config_minor_version) || + (xml_micro_version != $xml_config_micro_version)) + { + printf("*** libxml header files (version %d.%d.%d) do not match\n", + xml_major_version, xml_minor_version, xml_micro_version); + printf("*** xml2-config (version %d.%d.%d)\n", + $xml_config_major_version, $xml_config_minor_version, $xml_config_micro_version); + return 1; + } +/* Compare the headers to the library to make sure we match */ + /* Less than ideal -- doesn't provide us with return value feedback, + * only exits if there's a serious mismatch between header and library. + */ + LIBXML_TEST_VERSION; + + /* Test that the library is greater than our minimum version */ + if ((xml_major_version > major) || + ((xml_major_version == major) && (xml_minor_version > minor)) || + ((xml_major_version == major) && (xml_minor_version == minor) && + (xml_micro_version >= micro))) + { + return 0; + } + else + { + printf("\n*** An old version of libxml (%d.%d.%d) was found.\n", + xml_major_version, xml_minor_version, xml_micro_version); + printf("*** You need a version of libxml newer than %d.%d.%d. The latest version of\n", + major, minor, micro); + printf("*** libxml is always available from ftp://ftp.xmlsoft.org.\n"); + printf("***\n"); + printf("*** If you have already installed a sufficiently new version, this error\n"); + printf("*** probably means that the wrong copy of the xml2-config shell script is\n"); + printf("*** being found. The easiest way to fix this is to remove the old version\n"); + printf("*** of LIBXML, but you can also set the XML2_CONFIG environment to point to the\n"); + printf("*** correct copy of xml2-config. (In this case, you will have to\n"); + printf("*** modify your LD_LIBRARY_PATH enviroment variable, or edit /etc/ld.so.conf\n"); + printf("*** so that the correct libraries are found at run-time))\n"); + } + return 1; +} +],, no_xml=yes,[echo $ac_n "cross compiling; assumed OK... $ac_c"]) + CPPFLAGS="$ac_save_CPPFLAGS" + LIBS="$ac_save_LIBS" + fi + fi + + if test "x$no_xml" = x ; then + AC_MSG_RESULT(yes (version $xml_config_major_version.$xml_config_minor_version.$xml_config_micro_version)) + ifelse([$2], , :, [$2]) + else + AC_MSG_RESULT(no) + if test "$XML2_CONFIG" = "no" ; then + echo "*** The xml2-config script installed by LIBXML could not be found" + echo "*** If libxml was installed in PREFIX, make sure PREFIX/bin is in" + echo "*** your path, or set the XML2_CONFIG environment variable to the" + echo "*** full path to xml2-config." + else + if test -f conf.xmltest ; then + : + else + echo "*** Could not run libxml test program, checking why..." + CPPFLAGS="$CPPFLAGS $XML_CPPFLAGS" + LIBS="$LIBS $XML_LIBS" + AC_TRY_LINK([ +#include <libxml/xmlversion.h> +#include <stdio.h> +], [ LIBXML_TEST_VERSION; return 0;], + [ echo "*** The test program compiled, but did not run. This usually means" + echo "*** that the run-time linker is not finding LIBXML or finding the wrong" + echo "*** version of LIBXML. If it is not finding LIBXML, you'll need to set your" + echo "*** LD_LIBRARY_PATH environment variable, or edit /etc/ld.so.conf to point" + echo "*** to the installed location Also, make sure you have run ldconfig if that" + echo "*** is required on your system" + echo "***" + echo "*** If you have an old version installed, it is best to remove it, although" + echo "*** you may also be able to get things to work by modifying LD_LIBRARY_PATH" ], + [ echo "*** The test program failed to compile or link. See the file config.log for the" + echo "*** exact error that occured. This usually means LIBXML was incorrectly installed" + echo "*** or that you have moved LIBXML since it was installed. In the latter case, you" + echo "*** may want to edit the xml2-config script: $XML2_CONFIG" ]) + CPPFLAGS="$ac_save_CPPFLAGS" + LIBS="$ac_save_LIBS" + fi + fi + + XML_CPPFLAGS="" + XML_LIBS="" + ifelse([$3], , :, [$3]) + fi + AC_SUBST(XML_CPPFLAGS) + AC_SUBST(XML_LIBS) + rm -f conf.xmltest +]) + # Copyright (C) 2002, 2003, 2005, 2006, 2007, 2008 Free Software Foundation, Inc. # # This file is free software; the Free Software Foundation diff --git a/configure.ac b/configure.ac index 3a0effa..ac31bd0 100644 --- a/configure.ac +++ b/configure.ac @@ -5,6 +5,18 @@ AC_CHECK_LIB([virt], [virConnectOpen], [], AC_MSG_ERROR([You need libvirt to com AM_INIT_AUTOMAKE([-Wno-portability]) AM_MAINTAINER_MODE([enable]) +dnl Checks for programs. +AC_PROG_CC +AC_LANG([C]) +AC_PROG_INSTALL +AC_PROG_SED +AC_CHECK_TOOL([STRIP], [strip]) +AC_CHECK_TOOL([ECHO], [echo]) +AC_CHECK_TOOL([RM], [rm]) + +dnl Check for libxml2 +AM_PATH_XML2([2.0.0]) + dnl Specific XML catalog file for validation of generated html AC_ARG_WITH([xml-catalog-file], [AC_HELP_STRING([--with-xml-catalog-file=path], diff --git a/src/Makefile.am b/src/Makefile.am index 67a0192..da3ffab 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -1,32 +1,24 @@ -NAME=libvirt-php -CC=cc -CP=cp -RM=rm -ECHO=echo -PHPINC=`php-config --includes` -PHPEDIR=`php-config --extension-dir` -PHPCDIR=`php-config --configure-options | sed -n 's|.*--with-config-file-scan-dir=\([^ ]*\).*|\1|p'` -XMLCFLAGS=`pkg-config libxml-2.0 --cflags` -XMLLIBS=`pkg-config libxml-2.0 --libs` +PHPINC=$(shell php-config --includes) +PHPEDIR=$(shell php-config --extension-dir) +PHPCDIR=$(shell php-config --configure-options | sed -n 's|.*--with-config-file-scan-dir=\([^ ]*\).*|\1|p') EXTRA_DIST = libvirt.c php_libvirt.h all: build clean-temp build: - $(CC) -Wall -fpic -DCOMPILE_DL_LIBVIRT=1 $(PHPINC) -c -o $(NAME).o libvirt.c $(XMLCFLAGS) - $(CC) -Wall -shared $(LIBS) -rdynamic -o $(NAME).so $(NAME).o -ldl -lvirt $(XMLLIBS) - $(ECHO) "Extension compiled as $(NAME).so" + $(CC) -Wall -fpic -DCOMPILE_DL_LIBVIRT=1 $(PHPINC) -c -o $(PACKAGE).o libvirt.c $(XML_CPPFLAGS) + $(CC) -Wall -shared $(LIBS) -rdynamic -o $(PACKAGE).so $(PACKAGE).o -ldl -lvirt $(XML_LIBS) + $(ECHO) "Extension compiled as $(PACKAGE).so" install-exec-local: - mkdir -p $(DESTDIR)$(PHPEDIR) - $(CP) $(NAME).so $(DESTDIR)$(PHPEDIR) - mkdir -p $(DESTDIR)$(PHPCDIR) - $(ECHO) "extension=$(NAME).so" > $(DESTDIR)$(PHPCDIR)/libvirt-php.ini + $(INSTALL) -m 644 -D --strip $(PACKAGE).so $(DESTDIR)$(PHPEDIR)/$(PACKAGE).so + $(INSTALL) -m 755 -d $(DESTDIR)$(PHPCDIR) + $(ECHO) "extension=$(PACKAGE).so" > $(DESTDIR)$(PHPCDIR)/libvirt-php.ini uninstall-local: - $(RM) -f $(DESTDIR)$(PHPCDIR)/$(NAME).ini - $(RM) -f $(DESTDIR)$(PHPEDIR)/$(NAME).so + $(RM) -f $(DESTDIR)$(PHPCDIR)/$(PACKAGE).ini + $(RM) -f $(DESTDIR)$(PHPEDIR)/$(PACKAGE).so clean-temp: $(RM) -f *.o -- 1.7.1

On Fri, Feb 18, 2011 at 03:51:32PM +0800, Lyre wrote:
* configure.ac: Added checkings for compiler, sed, install, and so on; Added AM_PATH_XML2 to check libxml2. * src/Makefile.am: dropped needless macros since it is done by autoconf; Use $INSTALL to install files rather than cp. --- aclocal.m4 | 189 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ configure.ac | 12 ++++ src/Makefile.am | 30 +++------ 3 files changed, 212 insertions(+), 19 deletions(-)
diff --git a/aclocal.m4 b/aclocal.m4 index a1fa316..982143e 100644 --- a/aclocal.m4 +++ b/aclocal.m4 @@ -19,6 +19,195 @@ You have another version of autoconf. It may work, but is not guaranteed to. If you have problems, you may need to regenerate the build system entirely. To do so, use the procedure documented by the package, typically `autoreconf'.])])
+# Configure paths for LIBXML2 +# Mike Hommey 2004-06-19 +# use CPPFLAGS instead of CFLAGS +# Toshio Kuratomi 2001-04-21 +# Adapted from: +# Configure paths for GLIB +# Owen Taylor 97-11-3 + +dnl AM_PATH_XML2([MINIMUM-VERSION, [ACTION-IF-FOUND [, ACTION-IF-NOT-FOUND]]]) +dnl Test for XML, and define XML_CPPFLAGS and XML_LIBS +dnl
[snip] This code looks rather overly long & complex. I'd suggest taking the libxml2 check from libvirt's configure.ac file & using that instead. Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|

于 2011年02月18日 18:45, Daniel P. Berrange 写道:
On Fri, Feb 18, 2011 at 03:51:32PM +0800, Lyre wrote:
* configure.ac: Added checkings for compiler, sed, install, and so on; Added AM_PATH_XML2 to check libxml2. * src/Makefile.am: dropped needless macros since it is done by autoconf; Use $INSTALL to install files rather than cp. --- aclocal.m4 | 189 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ configure.ac | 12 ++++ src/Makefile.am | 30 +++------ 3 files changed, 212 insertions(+), 19 deletions(-)
diff --git a/aclocal.m4 b/aclocal.m4 index a1fa316..982143e 100644 --- a/aclocal.m4 +++ b/aclocal.m4 @@ -19,6 +19,195 @@ You have another version of autoconf. It may work, but is not guaranteed to. If you have problems, you may need to regenerate the build system entirely. To do so, use the procedure documented by the package, typically `autoreconf'.])])
+# Configure paths for LIBXML2 +# Mike Hommey 2004-06-19 +# use CPPFLAGS instead of CFLAGS +# Toshio Kuratomi 2001-04-21 +# Adapted from: +# Configure paths for GLIB +# Owen Taylor 97-11-3 + +dnl AM_PATH_XML2([MINIMUM-VERSION, [ACTION-IF-FOUND [, ACTION-IF-NOT-FOUND]]]) +dnl Test for XML, and define XML_CPPFLAGS and XML_LIBS +dnl
[snip]
This code looks rather overly long & complex. I'd suggest taking the libxml2 check from libvirt's configure.ac file & using that instead.
Daniel
Thanks! I'll take a look at it.

On 02/18/2011 08:51 AM, Lyre wrote:
Hi all:
This patch updeted the building system.
* Added necessary checkings in configure.ac * Cleared up libvirt-php.spec and src/Makefile.am
BTW, I've changed my email to business address.
Noted. Also, reviewed it and it's looking fine so I pushed this series. Thanks for rewriting the src/Makefile.am and adding checking for tools and libraries since this is looking much better with your patch. Thanks again, Michal -- Michal Novotny<minovotn@redhat.com>, RHCE Virtualization Team (xen userspace), Red Hat
participants (4)
-
Daniel P. Berrange
-
Lyre
-
Michal Novotny
-
李勇