[gvfs] Get rid of unnecessary AC_SUBST() calls



commit 475b1e1a05c9cb172fd9c354fef9fba2aa3381a6
Author: Benjamin Otte <otte gnome org>
Date:   Mon Nov 2 16:32:51 2009 +0100

    Get rid of unnecessary AC_SUBST() calls
    
    PKG_CHECK_MODULES() calls AC_SUBST for LIBS and CFLAGS automatically.
    
    Also name EXPAT variables properly - I got confused by LIBXML vs XML
    flags.

 configure.ac       |   41 ++++-------------------------------------
 daemon/Makefile.am |    4 ++--
 2 files changed, 6 insertions(+), 39 deletions(-)
---
diff --git a/configure.ac b/configure.ac
index 2d032fa..a8b3d61 100644
--- a/configure.ac
+++ b/configure.ac
@@ -43,12 +43,8 @@ DISTCHECK_CONFIGURE_FLAGS="--enable-gtk-doc"
 AC_SUBST(DISTCHECK_CONFIGURE_FLAGS)
 
 PKG_CHECK_MODULES(GLIB, glib-2.0 >= 2.21.2 gthread-2.0 gobject-2.0 gmodule-no-export-2.0 gio-unix-2.0 gio-2.0 )
-AC_SUBST(GLIB_CFLAGS)
-AC_SUBST(GLIB_LIBS)
 
 PKG_CHECK_MODULES(DBUS, dbus-1)
-AC_SUBST(DBUS_CFLAGS)
-AC_SUBST(DBUS_LIBS)
 
 AC_ARG_WITH(dbus_service_dir, [  --with-dbus-service-dir=PATH choose directory for dbus service files, [default=PREFIX/share/dbus-1/services]], with_dbus_service_dir="$withval", with_dbus_service_dir=$datadir/dbus-1/services)
 DBUS_SERVICE_DIR=$with_dbus_service_dir
@@ -158,8 +154,6 @@ PKG_CHECK_MODULES(LIBXML, libxml-2.0,
 	AC_DEFINE(HAVE_LIBXML, [], [Set if we have libxml])]
 	msg_libxml=yes,
         [AM_CONDITIONAL(HAVE_LIBXML, false)])
-AC_SUBST(LIBXML_CFLAGS)
-AC_SUBST(LIBXML_LIBS)
 
 dnl *************************
 dnl *** Check for libudev ***
@@ -178,9 +172,6 @@ if test "x$enable_udev" != "xno"; then
   fi
 fi
 
-AC_SUBST(UDEV_CFLAGS)
-AC_SUBST(UDEV_LIBS)
-
 dnl **********************
 dnl *** Check for FUSE ***
 dnl **********************
@@ -198,9 +189,6 @@ if test "x$enable_fuse" != "xno"; then
   fi
 fi
 
-AC_SUBST(FUSE_LIBS)
-AC_SUBST(FUSE_CFLAGS)
-
 AM_CONDITIONAL(USE_FUSE, [test "$msg_fuse" = "yes"])
 
 dnl **********************
@@ -220,9 +208,6 @@ if test "x$enable_gconf" != "xno"; then
   fi
 fi
 
-AC_SUBST(GCONF_LIBS)
-AC_SUBST(GCONF_CFLAGS)
-
 AM_CONDITIONAL(USE_GCONF, [test "$msg_gconf" = "yes"])
 
 dnl ************************************
@@ -244,9 +229,6 @@ if test "x$enable_gdu" != "xno"; then
   fi
 fi
 
-AC_SUBST(GDU_LIBS)
-AC_SUBST(GDU_CFLAGS)
-
 AM_CONDITIONAL(USE_GDU, [test "$msg_gdu" = "yes"])
 
 dnl **********************
@@ -275,9 +257,6 @@ if test "x$enable_hal" != "xno"; then
   fi
 fi
 
-AC_SUBST(HAL_LIBS)
-AC_SUBST(HAL_CFLAGS)
-
 AM_CONDITIONAL(USE_HAL, [test "$msg_hal" = "yes"])
 
 dnl **********************
@@ -295,8 +274,6 @@ if test "x$enable_gudev" != "xno"; then
     AC_DEFINE(HAVE_GUDEV, 1, [Define to 1 if gudev is available])
   fi
 fi
-AC_SUBST(GUDEV_CFLAGS)
-AC_SUBST(GUDEV_LIBS)
 AM_CONDITIONAL(USE_GUDEV, [test "$msg_gudev" = "yes"])
 
 dnl **************************************************
@@ -316,9 +293,6 @@ if test "x$enable_cdda" != "xno" -a \( "x$msg_hal" = "xyes" -o "x$msg_gudev" = "
   fi
 fi
 
-AC_SUBST(CDDA_LIBS)
-AC_SUBST(CDDA_CFLAGS)
-
 AM_CONDITIONAL(USE_CDDA, [test "$msg_cdda" = "yes"])
 
 dnl *************************************************
@@ -366,16 +340,15 @@ if test "x$enable_obexftp" != "xno";  then
 
     msg_obexftp=yes
     AC_DEFINE(HAVE_OBEXFTP, 1, [Define to 1 if ObexFTP is going to be built])
-    XML_CFLAGS=""
-    XML_LIBS="-lexpat"
+    EXPAT_CFLAGS=""
+    EXPAT_LIBS="-lexpat"
   else
     msg_obexftp=no
   fi
 fi
 
-AC_SUBST(XML_LIBS)
-AC_SUBST(XML_CFLAGS)
-
+AC_SUBST(EXPAT_CFLAGS)
+AC_SUBST(EXPAT_LIBS)
 AM_CONDITIONAL(USE_OBEXFTP, [test "$msg_obexftp" = "yes"])
 
 dnl *************************
@@ -410,9 +383,6 @@ if test "x$enable_gphoto2" != "xno" -a \( "x$msg_hal" = "xyes" -o "x$msg_gudev"
   fi
 fi
 
-AC_SUBST(GPHOTO2_LIBS)
-AC_SUBST(GPHOTO2_CFLAGS)
-
 AM_CONDITIONAL(USE_GPHOTO2, [test "$msg_gphoto2" = "yes"])
 
 dnl *******************************
@@ -432,9 +402,6 @@ if test "x$enable_keyring" != "xno"; then
   fi
 fi
 
-AC_SUBST(KEYRING_LIBS)
-AC_SUBST(KEYRING_CFLAGS)
-
 AM_CONDITIONAL(USE_KEYRING, [test "$msg_keyring" = "yes"])
 
 dnl ==========================================================================
diff --git a/daemon/Makefile.am b/daemon/Makefile.am
index 5de92f8..0879c78 100644
--- a/daemon/Makefile.am
+++ b/daemon/Makefile.am
@@ -10,7 +10,7 @@ INCLUDES =					\
 	-I$(top_srcdir)/common			\
 	-I$(top_builddir)			\
 	$(GLIB_CFLAGS) $(DBUS_CFLAGS) 		\
-	$(OBEXFTP_CFLAGS) $(XML_CFLAGS)		\
+	$(OBEXFTP_CFLAGS) $(EXPAT_CFLAGS)	\
 	$(KEYRING_CFLAGS)			\
 	-DDBUS_API_SUBJECT_TO_CHANGE		\
 	-DG_UDEV_API_IS_SUBJECT_TO_CHANGE	\
@@ -239,7 +239,7 @@ if USE_HAL
 gvfsd_obexftp_CPPFLAGS += $(HAL_CFLAGS)
 endif
 
-gvfsd_obexftp_LDADD = $(OBEXFTP_LIBS) $(XML_LIBS) $(libraries)
+gvfsd_obexftp_LDADD = $(OBEXFTP_LIBS) $(EXPAT_LIBS) $(libraries)
 if USE_HAL
 gvfsd_obexftp_LDADD += $(HAL_LIBS)
 endif



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