GNOME.org
 

[tracker/tracker-0.8] configure: Fix inconsistencies between miners

commit f0cfb7db3a71f28947a80c7f5215c535c028cdbd
Author: Martyn Russell <martyn lanedo com>
Date:   Tue May 11 16:26:28 2010 +0100

    configure: Fix inconsistencies between miners
    
    Use the same terminology HAVE_MINER_* everywhere

 Makefile.am             |    2 +-
 configure.ac            |  110 +++++++++++++++++++++-------------------------
 data/Makefile.am        |    2 +-
 data/dbus/Makefile.am   |    2 +-
 data/miners/Makefile.am |    4 +-
 src/miners/Makefile.am  |    2 +-
 src/plugins/Makefile.am |    4 +-
 7 files changed, 58 insertions(+), 68 deletions(-)
---
diff --git a/Makefile.am b/Makefile.am
index 63aa457..ef2e2c4 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -58,12 +58,12 @@ DISTCLEANFILES = 				\
 
 DISTCHECK_CONFIGURE_FLAGS =			\
 	--with-session-bus-services-dir="\$(datadir)"/dbus-1/services \
-	--disable-evolution-miner		\
 	--disable-nautilus-extension		\
 	--enable-unit-tests			\
 	--enable-functional-tests		\
 	--enable-gtk-doc			\
 	--enable-miner-rss			\
+	--disable-miner-evolution		\
 	--enable-unac				\
 	--enable-poppler-glib			\
 	--enable-exempi				\
diff --git a/configure.ac b/configure.ac
index 3cc32c1..f6d51a2 100644
--- a/configure.ac
+++ b/configure.ac
@@ -321,7 +321,7 @@ AM_CONDITIONAL(HAVE_LIBSTREAMANALYZER, test "$have_libstreamanalyzer" = "yes")
 dnl DBus services dir
 AC_ARG_WITH([session_bus_services_dir],
             AS_HELP_STRING([--with-session-bus-services-dir],
-	                   [Path to DBus services directory]))
+	                   [path to DBus services directory]))
 
 if test "x$with_session_bus_services_dir" = "x" ; then
    services_dir="$datadir/dbus-1/services"
@@ -722,7 +722,7 @@ fi
 
 AC_ARG_ENABLE(libinotify,
 	      AS_HELP_STRING([--enable-libinotify],
-			     [Enables libinotify for the filesystem monitoring [[default=yes]]]),,
+			     [enables libinotify for the filesystem monitoring [[default=yes]]]),,
 	      [enable_libinotify=yes])
 
 if test "x$enable_libinotify" != "xno"; then
@@ -824,17 +824,19 @@ AM_CONDITIONAL(HAVE_GNOME_KEYRING, test "x$have_gnome_keyring" = "xyes")
 # Miner Evolution
 ####################################################################
 
-AC_ARG_ENABLE([evolution_miner],
-              AS_HELP_STRING([--enable-evolution-miner],
-                             [enable support for Evolution data miner [[default=auto]]]),,
-              [enable_evolution_miner=auto])
+evolution_plugins_dir="/dev/null"
 
-if test "x$enable_evolution_miner" != "xno"; then
+AC_ARG_ENABLE([miner_evolution],
+              AS_HELP_STRING([--enable-miner-evolution],
+                             [enable Evolution email data miner [[default=auto]]]),,
+              [enable_miner_evolution=auto])
+
+if test "x$enable_miner_evolution" != "xno"; then
    PKG_CHECK_MODULES(EVOLUTION_PLUGIN, [
    		     evolution-plugin >= $EVO_REQUIRED
 		     evolution-data-server-1.2 >= $EDS_REQUIRED],
-		     have_evolution_plugin=yes,
-		     have_evolution_plugin=no)
+		     have_miner_evolution=yes,
+		     have_miner_evolution=no)
 
    PKG_CHECK_EXISTS([evolution-data-server-1.2 >= 2.29.1],
 		    [AC_DEFINE(HAVE_EDS_2_29_1, 1, [Define if we have eds 2.29.1 or newer])])
@@ -842,51 +844,47 @@ if test "x$enable_evolution_miner" != "xno"; then
    AC_SUBST(EVOLUTION_PLUGIN_CFLAGS)
    AC_SUBST(EVOLUTION_PLUGIN_LIBS)
 
-   if test "x$have_evolution_plugin" = "xyes"; then
-      dnl Evolution plugin will be compiled
-      enable_evolution_miner=yes
-
+   if test "x$have_miner_evolution" = "xyes"; then
       dnl Evolution plugins dir
       AC_ARG_WITH([evolution_plugins_dir],
                   AS_HELP_STRING([--with-evolution-plugins-dir],
-	                         [Path to Evolution plugins directory]))
+	                         [path to Evolution plugins directory]))
       if test "x$with_evolution_plugins_dir" = "x" ; then
          evolution_plugins_dir=`$PKG_CONFIG evolution-plugin --variable=plugindir`
       else
          evolution_plugins_dir="$with_evolution_plugins_dir"
       fi
-
-      EVOLUTION_PLUGIN_INSTALL_DIR=$evolution_plugins_dir
-   else
-      if test "x$enable_evolution_miner" = "xyes"; then
-        dnl If evolution-miner was explicitly enabled by the user, and
-        dnl  evolution-plugin deps not found, dump error
-        AC_MSG_ERROR([Couldn't find Evolution plugin requirements (evolution-dev, evolution-data-server-dev).])
-      else
-        dnl support 'auto' mode...
-        enable_evolution_miner="no  (disabled)"
-        EVOLUTION_PLUGIN_INSTALL_DIR=/dev/null
-      fi
    fi
-
-   AC_SUBST(EVOLUTION_PLUGIN_INSTALL_DIR)
 else
-   enable_evolution_miner="no  (disabled)"
+   have_miner_evolution="no  (disabled)"
+fi
+
+if test "x$enable_miner_evolution" = "xyes"; then
+   if test "x$have_miner_evolution" != "xyes"; then
+      AC_MSG_ERROR([Couldn't find evolution-dev and evolution-data-server-dev for Evolution miner.])
+   fi
 fi
 
-AM_CONDITIONAL(USING_EVOLUTION_MINER, test "x$enable_evolution_miner" = "xyes")
+EVOLUTION_PLUGIN_INSTALL_DIR=$evolution_plugins_dir
+AC_SUBST(EVOLUTION_PLUGIN_INSTALL_DIR)
+
+AM_CONDITIONAL(HAVE_MINER_EVOLUTION, test "x$have_miner_evolution" = "xyes")
 
 
 ####################################################################
 # Miner Kmail
 ####################################################################
 
-AC_ARG_ENABLE([kmail_miner],
-              AS_HELP_STRING([--enable-kmail-miner],
-                             [enable support for KMail data miner [[default=yes]]]),,
-              [enable_kmail_miner=yes])
+AC_ARG_ENABLE([miner_kmail],
+              AS_HELP_STRING([--enable-miner-kmail],
+                             [enable KMail email data miner [[default=yes]]]),,
+              [enable_miner_kmail=yes])
 
-AM_CONDITIONAL(USING_KMAIL_MINER, test "x$enable_kmail_miner" = "xyes")
+if test "x$enable_miner_kmail" = "xyes"; then
+   have_miner_kmail="yes"
+fi
+
+AM_CONDITIONAL(HAVE_MINER_KMAIL, test "x$enable_miner_kmail" = "xyes")
 
 
 ##################################################################
@@ -895,36 +893,28 @@ AM_CONDITIONAL(USING_KMAIL_MINER, test "x$enable_kmail_miner" = "xyes")
 
 AC_ARG_ENABLE([miner_rss],
               AS_HELP_STRING([--enable-miner-rss],
-                             [enable miner rss [[default=auto]]]),,
+                             [enable RSS data miner [[default=auto]]]),,
               [enable_miner_rss=auto])
 
 if test "x$enable_miner_rss" != "xno" ; then
    PKG_CHECK_MODULES(LIBGRSS,
                      [libgrss-0 >= $LIBGRSS_REQUIRED],
-                     [have_libgrss=yes],
-                     [have_libgrss=no])
-
-   AC_SUBST(LIBGRSS_CFLAGS)
-   AC_SUBST(LIBGRSS_LIBS)
-
-   if test "x$enable_miner_rss" = "xauto"; then
-      if  test "x$have_libgrss" = "xyes"; then
-          enable_miner_rss="yes"
-      else
-        enable_miner_rss="no"
-      fi
-   fi
+                     [have_miner_rss=yes],
+                     [have_miner_rss=no])
 
-   if test "x$enable_miner_rss" = "xyes"; then
-      if test "x$have_libgrss" != "xyes"; then
-         AC_MSG_ERROR([Couldn't find libgrss >= $LIBGRSS_REQUIRED. and miner rss requested])
-      fi
-   fi
+   AC_SUBST(MINER_RSS_CFLAGS)
+   AC_SUBST(MINER_RSS_LIBS)
 else
    enable_miner_rss="no  (disabled)"
 fi
 
-AM_CONDITIONAL(USING_MINER_RSS, test "x$enable_miner_rss" = "xyes")
+if test "x$enable_miner_rss" = "xyes"; then
+   if test "x$have_miner_rss" != "xyes"; then
+      AC_MSG_ERROR([Couldn't find libgrss >= $LIBGRSS_REQUIRED for RSS miner.])
+   fi
+fi
+
+AM_CONDITIONAL(HAVE_MINER_RSS, test "x$have_miner_rss" = "xyes")
 
 ####################################################################
 # Application and Vala requirements
@@ -1515,7 +1505,7 @@ AM_CONDITIONAL(HAVE_MOCKUP, test "x$have_mockup" = "xyes")
 
 AC_ARG_ENABLE([nautilus-extension],
               AS_HELP_STRING([--enable-nautilus-extension],
-                             [Enable the nautilus extension [[default=auto]]]),,
+                             [enable the nautilus extension [[default=auto]]]),,
               [enable_nautilus_extension=auto])
 
 if test "x$enable_nautilus_extension" != "xno"; then
@@ -1530,7 +1520,7 @@ if test "x$enable_nautilus_extension" != "xno"; then
    if test "x$have_nautilus_extension" = "xyes"; then
       AC_ARG_WITH([nautilus-extensions-dir],
                   AS_HELP_STRING([--with-nautilus-extensions-dir],
-                                 [Path to Nautilus extensions directory]))
+                                 [path to Nautilus extensions directory]))
 
       if test "x$with_nautilus_extensions_dir" = "x"; then
          nautilus_extensions_dir=`$PKG_CONFIG --variable=extensiondir libnautilus-extension`
@@ -1806,9 +1796,9 @@ Metadata Extractors:
 
 Data Miners:
 
-        RSS:                                    $enable_miner_rss
-        Evolution:                              $enable_evolution_miner
-        KMail:                                  $enable_kmail_miner
+        RSS:                                    $have_miner_rss
+        Evolution:                              $have_miner_evolution ($evolution_plugins_dir)
+        KMail:                                  $have_miner_kmail
 
 Plugins:
 
diff --git a/data/Makefile.am b/data/Makefile.am
index 46a3990..e7bb6bc 100644
--- a/data/Makefile.am
+++ b/data/Makefile.am
@@ -42,7 +42,7 @@ if HAVE_TRACKER_STATUS_ICON
 autostart_DATA += tracker-status-icon.desktop
 endif
 
-if USING_MINER_RSS
+if HAVE_MINER_RSS
 autostart_DATA += tracker-miner-rss.desktop
 endif
 
diff --git a/data/dbus/Makefile.am b/data/dbus/Makefile.am
index 25fd6b9..ca76836 100644
--- a/data/dbus/Makefile.am
+++ b/data/dbus/Makefile.am
@@ -24,7 +24,7 @@ service_in_files =						\
 	org.freedesktop.Tracker1.Miner.Files.service.in		\
 	org.freedesktop.Tracker1.Extract.service.in
 
-if USING_MINER_RSS
+if HAVE_MINER_RSS
 service_in_files += 						\
 	org.freedesktop.Tracker1.Miner.RSS.service.in
 endif
diff --git a/data/miners/Makefile.am b/data/miners/Makefile.am
index e768375..d0b331c 100644
--- a/data/miners/Makefile.am
+++ b/data/miners/Makefile.am
@@ -11,8 +11,8 @@ tracker_miners_DATA = \
 	tracker-miner-applications.desktop \
 	tracker-miner-files.desktop
 
-if USING_MINER_RSS
-tracker_miners_DATA += 	tracker-miner-rss.desktop
+if HAVE_MINER_RSS
+tracker_miners_DATA += tracker-miner-rss.desktop
 endif
 
 @INTLTOOL_DESKTOP_RULE@
diff --git a/src/miners/Makefile.am b/src/miners/Makefile.am
index f6373b6..74551bd 100644
--- a/src/miners/Makefile.am
+++ b/src/miners/Makefile.am
@@ -2,6 +2,6 @@ include $(top_srcdir)/Makefile.decl
 
 SUBDIRS = fs
 
-if USING_MINER_RSS
+if HAVE_MINER_RSS
 SUBDIRS += rss
 endif
diff --git a/src/plugins/Makefile.am b/src/plugins/Makefile.am
index fb96a68..a24c9d4 100644
--- a/src/plugins/Makefile.am
+++ b/src/plugins/Makefile.am
@@ -2,11 +2,11 @@ include $(top_srcdir)/Makefile.decl
 
 SUBDIRS =
 
-if USING_KMAIL_MINER
+if HAVE_MINER_KMAIL
 SUBDIRS += kmail
 endif
 
-if USING_EVOLUTION_MINER
+if HAVE_MINER_EVOLUTION
 SUBDIRS += evolution
 endif
[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]