From 64079f83528edb4cc4ae1a85a8633fd578b6e393 Mon Sep 17 00:00:00 2001 From: Vijaykumar Koppad Date: Wed, 19 Feb 2014 18:51:26 +0530 Subject: build: fail configure if the libxml2 devel package is missing. Problem: Geo-replication uses volume info xml output. While installing glusterfs through build+install from source, and if the libxml2-devel package is not present in the system, build will succeed with just warning and XML output : no. If user fails to observe this and doesn't know that geo-rep is dependent on xml output, user won't be able use geo-rep. Solution: If libxml2-devel is unavailable, and if geo-rep is enabled error out in configure, otherwise just warn with XML_OUTPUT = no. If the xml-output is disabled while configuring, error out if geo-rep is enabled, otherwise just warn. Change-Id: Icde5d0c1e5f3b869e7db6ed6a046bdc49b450fcb BUG: 1066997 Signed-off-by: Vijaykumar Koppad Reviewed-on: http://review.gluster.org/7131 Reviewed-by: Niels de Vos Tested-by: Gluster Build System --- configure.ac | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) (limited to 'configure.ac') diff --git a/configure.ac b/configure.ac index 7c383532e..69d25e4da 100644 --- a/configure.ac +++ b/configure.ac @@ -577,10 +577,18 @@ if test "x$enable_xml_output" != "xno"; then if test "x${no_xml}" = "x"; then AC_DEFINE([HAVE_LIB_XML], [1], [Define to 1 if using libxml2.]) else - AC_MSG_WARN([libxml2 devel libraries not found disabling XML support]) - BUILD_XML_OUTPUT="no" + if test "x$enable_georeplication" != "xno"; then + AC_MSG_ERROR([libxml2 devel libraries not found]) + else + AC_MSG_WARN([libxml2 devel libraries not found disabling XML support]) + BUILD_XML_OUTPUT="no" + fi + fi else + if test "x$enable_georeplication" != "xno"; then + AC_MSG_ERROR([geo-replication requires xml output]) + fi BUILD_XML_OUTPUT="no" fi # end of xml-output -- cgit