summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMartyn Russell <martyn@lanedo.com>2014-10-09 19:11:45 +0100
committerMartyn Russell <martyn@lanedo.com>2014-10-27 16:02:01 +0000
commit2b53cd5d3aaf046c413fd4afb188c16d7e611f48 (patch)
tree0b48c56334ed085c8ab6dd337144058800cd495a
parentdc4fbfe97d19d07559cad7cdef9b0ab2ec5bd319 (diff)
downloadtracker-2b53cd5d3aaf046c413fd4afb188c16d7e611f48.tar.gz
all: Migrate to GKeyfileSettingsBackend and generate man pages from schemas
Now we've removed the internal key file object, we've had to put in place another method for supporting the existing TRACKER_USE_CONFIG_FILES environment variable. Thanks to the GKeyfileSettingsBackend provided by GLib, we can fallback to old school INI type config files for embedded solutions or cases where we don't want dconf as a backend. This works rather well. IT should be noted, the INI files are *NOT* written out in full if they do not exist, only options which are saved or different to the default settings are. This is how it should be too. Now we build man pages based on GSettings schemas using xsltproc with the template in docs/manpages/gsettings.xsl. This is a useful aid when trying to understand what config files can have in them. One thing it does highlight, is the config documentation could be better :)
-rw-r--r--configure.ac27
-rw-r--r--docs/manpages/Makefile.am3
-rw-r--r--docs/manpages/gsettings.xsl54
-rw-r--r--src/libtracker-data/.gitignore1
-rw-r--r--src/libtracker-data/Makefile.am22
-rw-r--r--src/libtracker-data/tracker-db-config.c61
-rw-r--r--src/libtracker-fts/.gitignore1
-rw-r--r--src/libtracker-fts/Makefile.am25
-rw-r--r--src/libtracker-fts/tracker-fts-config.c64
-rw-r--r--src/miners/fs/.gitignore1
-rw-r--r--src/miners/fs/Makefile.am22
-rw-r--r--src/miners/fs/tracker-config.c32
-rw-r--r--src/tracker-extract/.gitignore1
-rw-r--r--src/tracker-extract/Makefile.am22
-rw-r--r--src/tracker-extract/tracker-config.c88
-rw-r--r--src/tracker-extract/tracker-config.h4
-rw-r--r--src/tracker-store/.gitignore1
-rw-r--r--src/tracker-store/Makefile.am30
-rw-r--r--src/tracker-store/tracker-config.c79
-rw-r--r--src/tracker-writeback/.gitignore1
-rw-r--r--src/tracker-writeback/Makefile.am22
-rw-r--r--src/tracker-writeback/tracker-config.c62
22 files changed, 527 insertions, 96 deletions
diff --git a/configure.ac b/configure.ac
index 4842875cd..c80454e49 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1771,6 +1771,32 @@ fi
AM_CONDITIONAL(HAVE_LIBXML2, test "x$have_libxml2" = "xyes")
+##################################################################
+# Check for xsltproc, used to generate cfg man pages (5)
+##################################################################
+
+AC_ARG_ENABLE([cfg-man-pages],
+ AS_HELP_STRING([--enable-cfg-man-pages],
+ [enable man pages for config (.cfg) files [[default=yes]]]),
+ [enable_cfg_man_pages=$enableval],
+ [enable_cfg_man_pages=yes])
+
+if test "x$enable_cfg_man_pages" != "xno" ; then
+ # Check for graphviz if we are building gtk_doc
+ AC_PATH_PROG(XSLTPROC, xsltproc)
+ AC_SUBST(XSLTPROC)
+else
+ have_xsltproc="no (disabled)"
+fi
+
+if test "x$enable_cfg_man_pages" = "xyes"; then
+ if test -z "$XSLTPROC"; then
+ AC_MSG_ERROR([Couldn't find xsltproc, try installing libxslt.])
+ fi
+fi
+
+AM_CONDITIONAL(HAVE_XSLTPROC, test -n "$XSLTPROC")
+
####################################################################
# Check for tracker-extract: gstreamer/mplayer/etc
####################################################################
@@ -2719,6 +2745,7 @@ Build Configuration:
Win32: $native_win32
Enable gtk doc (for documentation): $enable_gtk_doc
+ Enable man pages for GSettings: $enable_cfg_man_pages
Enable functional tests: $enable_functional_tests
Enable unit tests: $have_unit_tests
diff --git a/docs/manpages/Makefile.am b/docs/manpages/Makefile.am
index 8cbb71d67..a3d76b06b 100644
--- a/docs/manpages/Makefile.am
+++ b/docs/manpages/Makefile.am
@@ -35,4 +35,5 @@ EXTRA_DIST = \
$(common) \
$(tp) \
$(tn) \
- $(tmrss)
+ $(tmrss) \
+ gsettings.xsl
diff --git a/docs/manpages/gsettings.xsl b/docs/manpages/gsettings.xsl
new file mode 100644
index 000000000..e975eddca
--- /dev/null
+++ b/docs/manpages/gsettings.xsl
@@ -0,0 +1,54 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<xsl:stylesheet version="1.0" xmlns:xsl="http://www.w3.org/1999/XSL/Transform">
+
+<xsl:output method="xml" omit-xml-declaration="yes" disable-output-escaping="yes"/>
+
+<xsl:template match="/">
+.TH <xsl:value-of select="$config-file"/> 5 "<xsl:value-of select="$current-date"/>" <xsl:value-of select="$version"/> "Tracker Manual"
+
+.SH NAME
+<xsl:value-of select="$config-file"/>
+
+.SH SYNOPSIS
+$HOME/.config/tracker/<xsl:value-of select="$config-file"/>
+
+.SH DESCRIPTION
+Tracker's configuration is built on top of GSettings, part of GLib. This means that there is a proper schema for configurations and they can be viewed (normally) using the \fBdconf-editor\fR tool.
+
+Tracker also allows switching from the GSettings database, used by most (cross) desktop applications, to a key/value formatted files (like Microsoft's INI format). To do this, the environment variable \fBTRACKER_USE_CONFIG_FILES\fR must be defined before running the application using that configuration.
+
+So where is this configuration? Well, normally they're stored in \fI$HOME/.config/tracker/\fR, however, default values are not stored to config files, only \fBdifferent\fR values are. This man page describes what keys and values can be used.
+
+See EXAMPLES for a general overview.
+
+.SH OPTIONS
+<xsl:for-each select="schemalist/schema/key">
+.TP
+\fB<xsl:value-of select="@name"/>\fR=<xsl:value-of select="translate(default, '(\[ | \])', '')" disable-output-escaping="yes"/>
+.nf
+
+<xsl:value-of select="description" disable-output-escaping="yes"/>
+
+<xsl:for-each select="range">
+Values range from <xsl:value-of select="@min"/> to <xsl:value-of select="@max"/>.
+</xsl:for-each>
+.fi
+
+</xsl:for-each>
+
+.SH EXAMPLES
+The top level group is "General". The default configuration (if saved to <xsl:value-of select="$config-file"/>), would look like:
+
+.nf
+ [General]
+ <xsl:for-each select="schemalist/schema/key">
+ <xsl:value-of select="@name"/>=<xsl:value-of select="translate(default, '(\[ | \])', '')" disable-output-escaping="yes"/>;
+ </xsl:for-each>
+.fi
+
+.SH SEE ALSO
+.BR <xsl:value-of select="translate($config-file, '.', '\n')"/>
+
+</xsl:template>
+
+</xsl:stylesheet>
diff --git a/src/libtracker-data/.gitignore b/src/libtracker-data/.gitignore
index 79d98d419..5845fe0a1 100644
--- a/src/libtracker-data/.gitignore
+++ b/src/libtracker-data/.gitignore
@@ -6,3 +6,4 @@ tracker-sparql-scanner.c
tracker-turtle-reader.c
*.xml
*.valid
+*.cfg.5
diff --git a/src/libtracker-data/Makefile.am b/src/libtracker-data/Makefile.am
index 6398da781..0e47bb241 100644
--- a/src/libtracker-data/Makefile.am
+++ b/src/libtracker-data/Makefile.am
@@ -16,6 +16,8 @@ AM_CPPFLAGS = \
-DSHAREDIR=\""$(datadir)"\" \
$(LIBTRACKER_DATA_CFLAGS)
+all: $(libtracker_data_LTLIBRARIES) $(man_MANS)
+
libtracker_datadir = $(libdir)/tracker-$(TRACKER_API_VERSION)
libtracker_data_LTLIBRARIES = libtracker-data.la
@@ -86,14 +88,30 @@ gsettings_SCHEMAS = org.freedesktop.Tracker.DB.gschema.xml
@GSETTINGS_RULES@
+if HAVE_XSLTPROC
+DATE=`date +'%B %Y'`
+man_MANS = tracker-db.cfg.5
+else
+man_MANS =
+endif
+
+%.cfg.5: $(gsettings_SCHEMAS)
+ $(AM_V_GEN)$(XSLTPROC) \
+ --stringparam version "${VERSION}" \
+ --stringparam current-date "${DATE}" \
+ --stringparam config-file "$(@:.cfg.5=.cfg)" \
+ $(top_srcdir)/docs/manpages/gsettings.xsl $^ > $@
+
# Dist / Clean rules for all data
BUILT_SOURCES = \
libtracker_data_la_vala.stamp
CLEANFILES = \
libtracker_data_la_vala.stamp \
- $(gsettings_SCHEMAS)
+ $(gsettings_SCHEMAS) \
+ $(man_MANS)
EXTRA_DIST = \
libtracker-data.vapi \
- $(gsettings_SCHEMAS:.xml=.xml.in)
+ $(gsettings_SCHEMAS:.xml=.xml.in) \
+ $(man_MANS)
diff --git a/src/libtracker-data/tracker-db-config.c b/src/libtracker-data/tracker-db-config.c
index eb3c65d8a..2b95f1532 100644
--- a/src/libtracker-data/tracker-db-config.c
+++ b/src/libtracker-data/tracker-db-config.c
@@ -1,5 +1,6 @@
/*
* Copyright (C) 2009, Nokia <ivan.frade@nokia.com>
+ * Copyright (C) 2014, Lanedo <martyn@lanedo.com>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@@ -25,10 +26,15 @@
#include <glib.h>
#include <gio/gio.h>
+#define G_SETTINGS_ENABLE_BACKEND
+#include <gio/gsettingsbackend.h>
+
+#include <libtracker-common/tracker-common.h>
+
#include "tracker-db-config.h"
-/* GKeyFile defines */
-#define GROUP_JOURNAL "Journal"
+#define CONFIG_SCHEMA "org.freedesktop.Tracker.DB"
+#define CONFIG_PATH "/org/freedesktop/tracker/db/"
/* Default values */
#define DEFAULT_JOURNAL_CHUNK_SIZE 50
@@ -139,22 +145,61 @@ config_finalize (GObject *object)
(G_OBJECT_CLASS (tracker_db_config_parent_class)->finalize) (object);
}
-
static void
config_constructed (GObject *object)
{
+ GSettings *settings;
+
(G_OBJECT_CLASS (tracker_db_config_parent_class)->constructed) (object);
- g_settings_delay (G_SETTINGS (object));
+ settings = G_SETTINGS (object);
+
+ if (G_LIKELY (!g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
+ g_settings_delay (settings);
+ }
+
+ g_settings_bind (settings, "journal-chunk-size", object, "journal-chunk-size", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
+ g_settings_bind (settings, "journal-rotate-destination", object, "journal-rotate-destination", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
}
TrackerDBConfig *
tracker_db_config_new (void)
{
- return g_object_new (TRACKER_TYPE_DB_CONFIG,
- "schema-id", "org.freedesktop.Tracker.DB",
- "path", "/org/freedesktop/tracker/db/",
- NULL);
+ TrackerDBConfig *config = NULL;
+
+ /* FIXME: should we unset GSETTINGS_BACKEND env var? */
+
+ if (G_UNLIKELY (g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
+ GSettingsBackend *backend;
+ gchar *filename;
+ gboolean need_to_save;
+
+ filename = g_build_filename (g_get_user_config_dir (), "tracker", "tracker-db.cfg", NULL);
+
+ need_to_save = g_file_test (filename, G_FILE_TEST_EXISTS) == FALSE;
+
+ backend = g_keyfile_settings_backend_new (filename, CONFIG_PATH, "General");
+ g_info ("Using config file '%s'", filename);
+ g_free (filename);
+
+ config = g_object_new (TRACKER_TYPE_DB_CONFIG,
+ "backend", backend,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
+ g_object_unref (backend);
+
+ if (need_to_save) {
+ g_info (" Config file does not exist, using default values...");
+ }
+ } else {
+ config = g_object_new (TRACKER_TYPE_DB_CONFIG,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
+ }
+
+ return config;
}
gboolean
diff --git a/src/libtracker-fts/.gitignore b/src/libtracker-fts/.gitignore
index c29f42faa..5af0ddf8c 100644
--- a/src/libtracker-fts/.gitignore
+++ b/src/libtracker-fts/.gitignore
@@ -1,2 +1,3 @@
*.xml
*.valid
+*.cfg.5
diff --git a/src/libtracker-fts/Makefile.am b/src/libtracker-fts/Makefile.am
index 322365ecf..e583c391c 100644
--- a/src/libtracker-fts/Makefile.am
+++ b/src/libtracker-fts/Makefile.am
@@ -4,6 +4,8 @@ AM_CPPFLAGS = \
-DNDEBUG \
$(LIBTRACKER_FTS_CFLAGS)
+all: $(noinst_LTLIBRARIES) $(man_MANS)
+
noinst_LTLIBRARIES = libtracker-fts.la
fts4_sources = \
@@ -65,6 +67,25 @@ gsettings_SCHEMAS = org.freedesktop.Tracker.FTS.gschema.xml
@GSETTINGS_RULES@
+if HAVE_XSLTPROC
+DATE=`date +'%B %Y'`
+man_MANS = tracker-fts.cfg.5
+else
+man_MANS =
+endif
+
+%.cfg.5: $(gsettings_SCHEMAS)
+ $(AM_V_GEN)$(XSLTPROC) \
+ --stringparam version "${VERSION}" \
+ --stringparam current-date "${DATE}" \
+ --stringparam config-file "$(@:.cfg.5=.cfg)" \
+ $(top_srcdir)/docs/manpages/gsettings.xsl $^ > $@
+
# Dist / Clean rules for all data
-CLEANFILES = $(gsettings_SCHEMAS)
-EXTRA_DIST += $(gsettings_SCHEMAS:.xml=.xml.in)
+CLEANFILES = \
+ $(gsettings_SCHEMAS) \
+ $(man_MANS)
+
+EXTRA_DIST += \
+ $(gsettings_SCHEMAS:.xml=.xml.in) \
+ $(man_MANS)
diff --git a/src/libtracker-fts/tracker-fts-config.c b/src/libtracker-fts/tracker-fts-config.c
index 34ebc619d..64abf2547 100644
--- a/src/libtracker-fts/tracker-fts-config.c
+++ b/src/libtracker-fts/tracker-fts-config.c
@@ -1,5 +1,6 @@
/*
* Copyright (C) 2009, Nokia <ivan.frade@nokia.com>
+ * Copyright (C) 2014, Lanedo <martyn@lanedo.com>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@@ -25,10 +26,15 @@
#include <glib.h>
#include <gio/gio.h>
+#define G_SETTINGS_ENABLE_BACKEND
+#include <gio/gsettingsbackend.h>
+
+#include <libtracker-common/tracker-common.h>
+
#include "tracker-fts-config.h"
-/* GKeyFile defines */
-#define GROUP_INDEXING "Indexing"
+#define CONFIG_SCHEMA "org.freedesktop.Tracker.FTS"
+#define CONFIG_PATH "/org/freedesktop/tracker/fts/"
/* Default values */
#define DEFAULT_MAX_WORD_LENGTH 30 /* 0->200 */
@@ -217,18 +223,62 @@ config_finalize (GObject *object)
static void
config_constructed (GObject *object)
{
+ GSettings *settings;
+
(G_OBJECT_CLASS (tracker_fts_config_parent_class)->constructed) (object);
- g_settings_delay (G_SETTINGS (object));
+ settings = G_SETTINGS (object);
+
+ if (G_LIKELY (!g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
+ g_settings_delay (settings);
+ }
+
+ g_settings_bind (settings, "max-word-length", object, "max-word-length", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
+ g_settings_bind (settings, "enable-stemmer", object, "enable-stemmer", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
+ g_settings_bind (settings, "enable-unaccent", object, "enable-unaccent", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
+ g_settings_bind (settings, "ignore-numbers", object, "ignore-numbers", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
+ g_settings_bind (settings, "ignore-stop-words", object, "ignore-stop-words", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
+ g_settings_bind (settings, "max-words-to-index", object, "max-words-to-index", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
}
TrackerFTSConfig *
tracker_fts_config_new (void)
{
- return g_object_new (TRACKER_TYPE_FTS_CONFIG,
- "schema-id", "org.freedesktop.Tracker.FTS",
- "path", "/org/freedesktop/tracker/fts/",
- NULL);
+ TrackerFTSConfig *config = NULL;
+
+ /* FIXME: should we unset GSETTINGS_BACKEND env var? */
+
+ if (G_UNLIKELY (g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
+ GSettingsBackend *backend;
+ gchar *filename;
+ gboolean need_to_save;
+
+ filename = g_build_filename (g_get_user_config_dir (), "tracker", "tracker-fts.cfg", NULL);
+
+ need_to_save = g_file_test (filename, G_FILE_TEST_EXISTS) == FALSE;
+
+ backend = g_keyfile_settings_backend_new (filename, CONFIG_PATH, "General");
+ g_info ("Using config file '%s'", filename);
+ g_free (filename);
+
+ config = g_object_new (TRACKER_TYPE_FTS_CONFIG,
+ "backend", backend,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
+ g_object_unref (backend);
+
+ if (need_to_save) {
+ g_info (" Config file does not exist, using default values...");
+ }
+ } else {
+ config = g_object_new (TRACKER_TYPE_FTS_CONFIG,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
+ }
+
+ return config;
}
gboolean
diff --git a/src/miners/fs/.gitignore b/src/miners/fs/.gitignore
index b554dc2bc..019715094 100644
--- a/src/miners/fs/.gitignore
+++ b/src/miners/fs/.gitignore
@@ -2,3 +2,4 @@ tracker-miner-fs
*.service
*.xml
*.valid
+*.cfg.5
diff --git a/src/miners/fs/Makefile.am b/src/miners/fs/Makefile.am
index ef3a30387..3d00786ba 100644
--- a/src/miners/fs/Makefile.am
+++ b/src/miners/fs/Makefile.am
@@ -7,6 +7,8 @@ AM_CPPFLAGS = \
-DLIBEXEC_PATH=\""$(libexecdir)"\" \
$(TRACKER_MINER_FS_CFLAGS)
+all: $(libexec_PROGRAMS) $(man_MANS)
+
libexec_PROGRAMS = tracker-miner-fs
# Prefer UPower for the power data if it's available, else fall back to HAL
@@ -97,17 +99,33 @@ gsettings_SCHEMAS = org.freedesktop.Tracker.Miner.Files.gschema.xml
@GSETTINGS_RULES@
+if HAVE_XSLTPROC
+DATE=`date +'%B %Y'`
+man_MANS = $(libexec_PROGRAMS).cfg.5
+else
+man_MANS =
+endif
+
+%.cfg.5: $(gsettings_SCHEMAS)
+ $(AM_V_GEN)$(XSLTPROC) \
+ --stringparam version "${VERSION}" \
+ --stringparam current-date "${DATE}" \
+ --stringparam config-file "$(@:.cfg.5=.cfg)" \
+ $(top_srcdir)/docs/manpages/gsettings.xsl $^ > $@
+
# Dist / Clean rules for all data
CLEANFILES = \
$(autostart_DATA) \
$(autostart_DATA:.desktop=.desktop.in) \
$(service_DATA) \
$(service_DATA:.service=.service.in) \
- $(gsettings_SCHEMAS)
+ $(gsettings_SCHEMAS) \
+ $(man_MANS)
EXTRA_DIST = \
$(autostart_DATA:.desktop=.desktop.in.in) \
$(config_DATA) \
$(service_DATA:.service=.service.in.in) \
- $(gsettings_SCHEMAS:.xml=.xml.in)
+ $(gsettings_SCHEMAS:.xml=.xml.in) \
+ $(man_MANS)
diff --git a/src/miners/fs/tracker-config.c b/src/miners/fs/tracker-config.c
index 3ffa922b4..733caa66e 100644
--- a/src/miners/fs/tracker-config.c
+++ b/src/miners/fs/tracker-config.c
@@ -1,5 +1,6 @@
/*
* Copyright (C) 2009, Nokia <ivan.frade@nokia.com>
+ * Copyright (C) 2014, Lanedo <martyn@lanedo.com>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public
@@ -693,10 +694,27 @@ config_constructed (GObject *object)
g_settings_delay (settings);
}
- /* Set up bindings */
- g_settings_bind (settings, "verbosity", object, "verbosity", G_SETTINGS_BIND_GET_NO_CHANGES);
+ /* Set up bindings:
+ *
+ * What's interesting here is that 'verbosity' and
+ * 'initial-sleep' are command line arguments that can be
+ * overridden, so we don't update the config when we set them
+ * from main() because it's a session configuration only, not
+ * a permanent one. To do this we use the flag
+ * G_SETTINGS_BIND_GET_NO_CHANGES.
+ *
+ * For the other settings, we don't bind the
+ * G_SETTINGS_BIND_SET because we don't want to save anything,
+ * ever, we only want to know about updates to the settings as
+ * they're changed externally. The only time this may be
+ * different is where we use the environment variable
+ * TRACKER_USE_CONFIG_FILES and we want to write a config
+ * file for convenience. But this is only necessary if the
+ * config is different to the default.
+ */
+ g_settings_bind (settings, "verbosity", object, "verbosity", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
g_settings_bind (settings, "sched-idle", object, "sched-idle", G_SETTINGS_BIND_GET);
- g_settings_bind (settings, "initial-sleep", object, "initial-sleep", G_SETTINGS_BIND_GET_NO_CHANGES);
+ g_settings_bind (settings, "initial-sleep", object, "initial-sleep", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
g_settings_bind (settings, "throttle", object, "throttle", G_SETTINGS_BIND_GET);
g_settings_bind (settings, "low-disk-space-limit", object, "low-disk-space-limit", G_SETTINGS_BIND_GET);
g_settings_bind (settings, "crawling-interval", object, "crawling-interval", G_SETTINGS_BIND_GET);
@@ -736,11 +754,10 @@ tracker_config_new (void)
need_to_save = g_file_test (filename, G_FILE_TEST_EXISTS) == FALSE;
- backend = g_keyfile_settings_backend_new (filename, CONFIG_PATH, NULL);
+ backend = g_keyfile_settings_backend_new (filename, CONFIG_PATH, "General");
g_info ("Using config file '%s'", filename);
g_free (filename);
- /* settings = g_settings_new_with_backend (CONFIG_SCHEME, backend); */
config = g_object_new (TRACKER_TYPE_CONFIG,
"backend", backend,
"schema-id", CONFIG_SCHEMA,
@@ -749,8 +766,7 @@ tracker_config_new (void)
g_object_unref (backend);
if (need_to_save) {
- g_info (" Config file did not exist, creating...");
- g_settings_apply (G_SETTINGS (config));
+ g_info (" Config file does not exist, using default values...");
}
} else {
config = g_object_new (TRACKER_TYPE_CONFIG,
@@ -932,7 +948,6 @@ tracker_config_set_verbosity (TrackerConfig *config,
{
g_return_if_fail (TRACKER_IS_CONFIG (config));
- /* g_object_set (G_OBJECT (config), "verbosity", value, NULL); */
g_settings_set_enum (G_SETTINGS (config), "verbosity", value);
g_object_notify (G_OBJECT (config), "verbosity");
}
@@ -943,7 +958,6 @@ tracker_config_set_initial_sleep (TrackerConfig *config,
{
g_return_if_fail (TRACKER_IS_CONFIG (config));
- /* g_object_set (G_OBJECT (config), "initial-sleep", value, NULL); */
g_settings_set_int (G_SETTINGS (config), "initial-sleep", value);
g_object_notify (G_OBJECT (config), "initial-sleep");
}
diff --git a/src/tracker-extract/.gitignore b/src/tracker-extract/.gitignore
index 7e3acbdf3..7681b5af5 100644
--- a/src/tracker-extract/.gitignore
+++ b/src/tracker-extract/.gitignore
@@ -5,3 +5,4 @@ tracker-extract-priority-dbus.h
*.service
*.xml
*.valid
+*.cfg.5
diff --git a/src/tracker-extract/Makefile.am b/src/tracker-extract/Makefile.am
index 6a0b58f61..ced5ef0c0 100644
--- a/src/tracker-extract/Makefile.am
+++ b/src/tracker-extract/Makefile.am
@@ -529,6 +529,8 @@ libextract_dummy_la_LIBADD = \
#
# Binaries
#
+all: $(libexec_PROGRAMS) $(man_MANS)
+
libexec_PROGRAMS = tracker-extract
tracker_extract_SOURCES = \
@@ -632,16 +634,32 @@ gsettings_SCHEMAS = org.freedesktop.Tracker.Extract.gschema.xml
@GSETTINGS_RULES@
+if HAVE_XSLTPROC
+DATE=`date +'%B %Y'`
+man_MANS = $(libexec_PROGRAMS).cfg.5
+else
+man_MANS =
+endif
+
+%.cfg.5: $(gsettings_SCHEMAS)
+ $(AM_V_GEN)$(XSLTPROC) \
+ --stringparam version "${VERSION}" \
+ --stringparam current-date "${DATE}" \
+ --stringparam config-file "$(@:.cfg.5=.cfg)" \
+ $(top_srcdir)/docs/manpages/gsettings.xsl $^ > $@
+
# Dist / Clean rules for all data
CLEANFILES += \
$(autostart_DATA) \
$(autostart_DATA:.desktop=.desktop.in) \
$(service_DATA) \
$(service_DATA:.service=.service.in) \
- $(gsettings_SCHEMAS)
+ $(gsettings_SCHEMAS) \
+ $(man_MANS)
EXTRA_DIST += \
$(autostart_DATA:.desktop=.desktop.in.in) \
$(config_DATA) \
$(service_DATA:.service=.service.in.in) \
- $(gsettings_SCHEMAS:.xml=.xml.in)
+ $(gsettings_SCHEMAS:.xml=.xml.in) \
+ $(man_MANS)
diff --git a/src/tracker-extract/tracker-config.c b/src/tracker-extract/tracker-config.c
index 04b0e2842..cc3063393 100644
--- a/src/tracker-extract/tracker-config.c
+++ b/src/tracker-extract/tracker-config.c
@@ -1,5 +1,6 @@
/*
* Copyright (C) 2009, Nokia <ivan.frade@nokia.com>
+ * Copyright (C) 2014, Lanedo <martyn@lanedo.com>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
@@ -26,13 +27,9 @@
#include "tracker-config.h"
-#define CONFIG_SCHEME "org.freedesktop.Tracker.Extract"
+#define CONFIG_SCHEMA "org.freedesktop.Tracker.Extract"
#define CONFIG_PATH "/org/freedesktop/tracker/extract/"
-typedef struct {
- GSettings *settings;
-} TrackerConfigPrivate;
-
static void config_set_property (GObject *object,
guint param_id,
const GValue *value,
@@ -46,7 +43,6 @@ static void config_constructed (GObject *object);
enum {
PROP_0,
- PROP_SETTINGS,
PROP_VERBOSITY,
PROP_SCHED_IDLE,
PROP_MAX_BYTES,
@@ -54,7 +50,7 @@ enum {
PROP_WAIT_FOR_MINER_FS,
};
-G_DEFINE_TYPE (TrackerConfig, tracker_config, G_TYPE_OBJECT);
+G_DEFINE_TYPE (TrackerConfig, tracker_config, G_TYPE_SETTINGS);
static void
tracker_config_class_init (TrackerConfigClass *klass)
@@ -66,14 +62,6 @@ tracker_config_class_init (TrackerConfigClass *klass)
object_class->finalize = config_finalize;
object_class->constructed = config_constructed;
- g_object_class_install_property (object_class,
- PROP_SETTINGS,
- g_param_spec_object ("settings",
- "GSettings",
- "GSettings GObject used",
- G_TYPE_SETTINGS,
- G_PARAM_WRITABLE | G_PARAM_CONSTRUCT_ONLY));
-
/* General */
g_object_class_install_property (object_class,
PROP_VERBOSITY,
@@ -118,14 +106,11 @@ tracker_config_class_init (TrackerConfigClass *klass)
"%TRUE to wait for tracker-miner-fs is done before extracting. %FAlSE otherwise",
FALSE,
G_PARAM_READWRITE));
-
- g_type_class_add_private (object_class, sizeof (TrackerConfigPrivate));
}
static void
tracker_config_init (TrackerConfig *object)
{
- object->priv = G_TYPE_INSTANCE_GET_PRIVATE (object, TRACKER_TYPE_CONFIG, TrackerConfigPrivate);
}
static void
@@ -137,14 +122,6 @@ config_set_property (GObject *object,
TrackerConfig *config = TRACKER_CONFIG (object);
switch (param_id) {
- case PROP_SETTINGS: {
- TrackerConfigPrivate *priv;
-
- priv = config->priv;
- priv->settings = g_value_get_object (value);
- break;
- }
-
/* General */
/* NOTE: We handle these because we have to be able
* to save these based on command line overrides.
@@ -219,22 +196,35 @@ config_finalize (GObject *object)
static void
config_constructed (GObject *object)
{
- TrackerConfigPrivate *priv;
GSettings *settings;
(G_OBJECT_CLASS (tracker_config_parent_class)->constructed) (object);
- priv = TRACKER_CONFIG (object)->priv;
- settings = priv->settings;
+ settings = G_SETTINGS (object);
- if (!settings) {
- priv->settings = settings = g_settings_new_with_path (CONFIG_SCHEME, CONFIG_PATH);
+ if (G_LIKELY (!g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
+ g_settings_delay (settings);
}
- g_settings_delay (settings);
-
- /* Set up bindings */
- g_settings_bind (settings, "verbosity", object, "verbosity", G_SETTINGS_BIND_GET_NO_CHANGES);
+ /* Set up bindings:
+ *
+ * What's interesting here is that 'verbosity' and
+ * 'initial-sleep' are command line arguments that can be
+ * overridden, so we don't update the config when we set them
+ * from main() because it's a session configuration only, not
+ * a permanent one. To do this we use the flag
+ * G_SETTINGS_BIND_GET_NO_CHANGES.
+ *
+ * For the other settings, we don't bind the
+ * G_SETTINGS_BIND_SET because we don't want to save anything,
+ * ever, we only want to know about updates to the settings as
+ * they're changed externally. The only time this may be
+ * different is where we use the environment variable
+ * TRACKER_USE_CONFIG_FILES and we want to write a config
+ * file for convenience. But this is only necessary if the
+ * config is different to the default.
+ */
+ g_settings_bind (settings, "verbosity", object, "verbosity", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
g_settings_bind (settings, "sched-idle", object, "sched-idle", G_SETTINGS_BIND_GET);
g_settings_bind (settings, "max-bytes", object, "max-bytes", G_SETTINGS_BIND_GET);
g_settings_bind (settings, "max-media-art-width", object, "max-media-art-width", G_SETTINGS_BIND_GET);
@@ -244,25 +234,43 @@ config_constructed (GObject *object)
TrackerConfig *
tracker_config_new (void)
{
- GSettings *settings = NULL;
+ TrackerConfig *config = NULL;
+
+ /* FIXME: should we unset GSETTINGS_BACKEND env var? */
if (G_UNLIKELY (g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
- /* FIXME: should we unset GSETTINGS_BACKEND env var? */
GSettingsBackend *backend;
gchar *filename, *basename;
+ gboolean need_to_save;
basename = g_strdup_printf ("%s.cfg", g_get_prgname ());
filename = g_build_filename (g_get_user_config_dir (), "tracker", basename, NULL);
g_free (basename);
- backend = g_keyfile_settings_backend_new (filename, CONFIG_PATH, NULL);
+ need_to_save = g_file_test (filename, G_FILE_TEST_EXISTS) == FALSE;
+
+ backend = g_keyfile_settings_backend_new (filename, CONFIG_PATH, "General");
+ g_info ("Using config file '%s'", filename);
g_free (filename);
- settings = g_settings_new_with_backend (CONFIG_SCHEME, backend);
+ config = g_object_new (TRACKER_TYPE_CONFIG,
+ "backend", backend,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
g_object_unref (backend);
+
+ if (need_to_save) {
+ g_info (" Config file does not exist, using default values...");
+ }
+ } else {
+ config = g_object_new (TRACKER_TYPE_CONFIG,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
}
- return g_object_new (TRACKER_TYPE_CONFIG, "settings", settings, NULL);
+ return config;
}
gint
diff --git a/src/tracker-extract/tracker-config.h b/src/tracker-extract/tracker-config.h
index c3503108f..daf157488 100644
--- a/src/tracker-extract/tracker-config.h
+++ b/src/tracker-extract/tracker-config.h
@@ -35,12 +35,12 @@ typedef struct TrackerConfig TrackerConfig;
typedef struct TrackerConfigClass TrackerConfigClass;
struct TrackerConfig {
- GObject parent;
+ GSettings parent;
gpointer priv;
};
struct TrackerConfigClass {
- GObjectClass parent_class;
+ GSettingsClass parent_class;
};
GType tracker_config_get_type (void) G_GNUC_CONST;
diff --git a/src/tracker-store/.gitignore b/src/tracker-store/.gitignore
index 4045ead9c..6d5b12ff9 100644
--- a/src/tracker-store/.gitignore
+++ b/src/tracker-store/.gitignore
@@ -11,3 +11,4 @@ tracker-store.h
*.service
*.xml
*.valid
+*.cfg.5
diff --git a/src/tracker-store/Makefile.am b/src/tracker-store/Makefile.am
index 1097e54ee..4c201de46 100644
--- a/src/tracker-store/Makefile.am
+++ b/src/tracker-store/Makefile.am
@@ -13,6 +13,8 @@ AM_CPPFLAGS = \
#
# Daemon sources
#
+all: $(libexec_PROGRAMS) $(man_MANS)
+
libexec_PROGRAMS = tracker-store
tracker_store_SOURCES = \
@@ -95,25 +97,41 @@ autostart_DATA = tracker-store.desktop
gsettings_ENUM_NAMESPACE = org.freedesktop.Tracker
gsettings_ENUM_FILES = $(top_srcdir)/src/libtracker-common/tracker-enums.h
-gsettings_SCHEMAS = \
- org.freedesktop.Tracker.gschema.xml \
- org.freedesktop.Tracker.Store.gschema.xml
+gsettings_SCHEMAS = $(schemalist) $(config)
+
+config = org.freedesktop.Tracker.Store.gschema.xml
+schemalist = org.freedesktop.Tracker.gschema.xml
@INTLTOOL_XML_NOMERGE_RULE@
@GSETTINGS_RULES@
+if HAVE_XSLTPROC
+DATE=`date +'%B %Y'`
+man_MANS = $(libexec_PROGRAMS).cfg.5
+else
+man_MANS =
+endif
+
+%.cfg.5: $(config)
+ $(AM_V_GEN)$(XSLTPROC) \
+ --stringparam version "${VERSION}" \
+ --stringparam current-date "${DATE}" \
+ --stringparam config-file "$(@:.cfg.5=.cfg)" \
+ $(top_srcdir)/docs/manpages/gsettings.xsl $^ > $@
+
# Dist / Clean rules for all data
CLEANFILES = \
$(autostart_DATA) \
$(autostart_DATA:.desktop=.desktop.in) \
$(service_DATA) \
- $(gsettings_SCHEMAS)
+ $(gsettings_SCHEMAS) \
+ $(man_MANS)
EXTRA_DIST += \
$(autostart_DATA:.desktop=.desktop.in.in) \
$(service_DATA:.service=.service.in) \
$(config_DATA) \
- $(gsettings_SCHEMAS:.xml=.xml.in)
-
+ $(gsettings_SCHEMAS:.xml=.xml.in) \
+ $(man_MANS)
diff --git a/src/tracker-store/tracker-config.c b/src/tracker-store/tracker-config.c
index 384c378fb..1c3f9c358 100644
--- a/src/tracker-store/tracker-config.c
+++ b/src/tracker-store/tracker-config.c
@@ -1,5 +1,6 @@
/*
* Copyright (C) 2009, Nokia <ivan.frade@nokia.com>
+ * Copyright (C) 2014, Lanedo <martyn@lanedo.com>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public
@@ -25,10 +26,16 @@
#include <glib.h>
#include <gio/gio.h>
+#define G_SETTINGS_ENABLE_BACKEND
+#include <gio/gsettingsbackend.h>
+
#include <libtracker-common/tracker-common.h>
#include "tracker-config.h"
+#define CONFIG_SCHEMA "org.freedesktop.Tracker.Store"
+#define CONFIG_PATH "/org/freedesktop/tracker/store/"
+
#define GRAPHUPDATED_DELAY_DEFAULT 1000
static void config_set_property (GObject *object,
@@ -142,18 +149,78 @@ config_finalize (GObject *object)
static void
config_constructed (GObject *object)
{
+ GSettings *settings;
+
(G_OBJECT_CLASS (tracker_config_parent_class)->constructed) (object);
- g_settings_delay (G_SETTINGS (object));
+ settings = G_SETTINGS (object);
+
+ if (G_LIKELY (!g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
+ g_settings_delay (settings);
+ }
+
+ /* Set up bindings:
+ *
+ * What's interesting here is that 'verbosity' and
+ * 'initial-sleep' are command line arguments that can be
+ * overridden, so we don't update the config when we set them
+ * from main() because it's a session configuration only, not
+ * a permanent one. To do this we use the flag
+ * G_SETTINGS_BIND_GET_NO_CHANGES.
+ *
+ * For the other settings, we don't bind the
+ * G_SETTINGS_BIND_SET because we don't want to save anything,
+ * ever, we only want to know about updates to the settings as
+ * they're changed externally. The only time this may be
+ * different is where we use the environment variable
+ * TRACKER_USE_CONFIG_FILES and we want to write a config
+ * file for convenience. But this is only necessary if the
+ * config is different to the default.
+ */
+ g_settings_bind (settings, "verbosity", object, "verbosity", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
+ g_settings_bind (settings, "graphupdated-delay", object, "graphupdated-delay", G_SETTINGS_BIND_GET);
}
TrackerConfig *
tracker_config_new (void)
{
- return g_object_new (TRACKER_TYPE_CONFIG,
- "schema-id", "org.freedesktop.Tracker.Store",
- "path", "/org/freedesktop/tracker/store/",
- NULL);
+ TrackerConfig *config = NULL;
+
+ /* FIXME: should we unset GSETTINGS_BACKEND env var? */
+
+ if (G_UNLIKELY (g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
+ GSettingsBackend *backend;
+ gchar *filename, *basename;
+ gboolean need_to_save;
+
+ basename = g_strdup_printf ("%s.cfg", g_get_prgname ());
+ filename = g_build_filename (g_get_user_config_dir (), "tracker", basename, NULL);
+ g_free (basename);
+
+ need_to_save = g_file_test (filename, G_FILE_TEST_EXISTS) == FALSE;
+
+ backend = g_keyfile_settings_backend_new (filename, CONFIG_PATH, "General");
+ g_info ("Using config file '%s'", filename);
+ g_free (filename);
+
+ config = g_object_new (TRACKER_TYPE_CONFIG,
+ "backend", backend,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
+ g_object_unref (backend);
+
+ if (need_to_save) {
+ g_info (" Config file does not exist, using default values...");
+ }
+ } else {
+ config = g_object_new (TRACKER_TYPE_CONFIG,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
+ }
+
+ return config;
}
gint
@@ -185,7 +252,7 @@ tracker_config_get_graphupdated_delay (TrackerConfig *config)
void
tracker_config_set_graphupdated_delay (TrackerConfig *config,
- gint value)
+ gint value)
{
g_return_if_fail (TRACKER_IS_CONFIG (config));
diff --git a/src/tracker-writeback/.gitignore b/src/tracker-writeback/.gitignore
index a6d2e3850..6e569e743 100644
--- a/src/tracker-writeback/.gitignore
+++ b/src/tracker-writeback/.gitignore
@@ -2,3 +2,4 @@ tracker-writeback
*.service
*.xml
*.valid
+*.cfg.5
diff --git a/src/tracker-writeback/Makefile.am b/src/tracker-writeback/Makefile.am
index 89959b8b5..55fb8a1c7 100644
--- a/src/tracker-writeback/Makefile.am
+++ b/src/tracker-writeback/Makefile.am
@@ -42,6 +42,8 @@ endif
#
# Binaries
#
+all: $(libexec_PROGRAMS) $(man_MANS)
+
libexec_PROGRAMS = tracker-writeback
tracker_writeback_SOURCES = \
@@ -83,12 +85,28 @@ gsettings_SCHEMAS = org.freedesktop.Tracker.Writeback.gschema.xml
@GSETTINGS_RULES@
+if HAVE_XSLTPROC
+DATE=`date +'%B %Y'`
+man_MANS = $(libexec_PROGRAMS).cfg.5
+else
+man_MANS =
+endif
+
+%.cfg.5: $(gsettings_SCHEMAS)
+ $(AM_V_GEN)$(XSLTPROC) \
+ --stringparam version "${VERSION}" \
+ --stringparam current-date "${DATE}" \
+ --stringparam config-file "$(@:.cfg.5=.cfg)" \
+ $(top_srcdir)/docs/manpages/gsettings.xsl $^ > $@
+
# Dist / Clean rules for all data
CLEANFILES = \
$(service_DATA) \
- $(gsettings_SCHEMAS)
+ $(gsettings_SCHEMAS) \
+ $(man_MANS)
EXTRA_DIST = \
$(service_DATA:.service=.service.in) \
$(config_DATA) \
- $(gsettings_SCHEMAS:.xml=.xml.in)
+ $(gsettings_SCHEMAS:.xml=.xml.in) \
+ $(man_MANS)
diff --git a/src/tracker-writeback/tracker-config.c b/src/tracker-writeback/tracker-config.c
index 59e7940de..dce879835 100644
--- a/src/tracker-writeback/tracker-config.c
+++ b/src/tracker-writeback/tracker-config.c
@@ -1,5 +1,6 @@
/*
* Copyright (C) 2009, Nokia <ivan.frade@nokia.com>
+ * Copyright (C) 2014, Lanedo <martyn@lanedo.com>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public
@@ -25,10 +26,16 @@
#include <glib.h>
#include <gio/gio.h>
+#define G_SETTINGS_ENABLE_BACKEND
+#include <gio/gsettingsbackend.h>
+
#include <libtracker-common/tracker-common.h>
#include "tracker-config.h"
+#define CONFIG_SCHEMA "org.freedesktop.Tracker.Writeback"
+#define CONFIG_PATH "/org/freedesktop/tracker/writeback/"
+
static void config_set_property (GObject *object,
guint param_id,
const GValue *value,
@@ -45,7 +52,7 @@ enum {
PROP_VERBOSITY,
};
-G_DEFINE_TYPE (TrackerConfig, tracker_config, G_TYPE_OBJECT);
+G_DEFINE_TYPE (TrackerConfig, tracker_config, G_TYPE_SETTINGS);
static void
tracker_config_class_init (TrackerConfigClass *klass)
@@ -76,7 +83,7 @@ static void
config_set_property (GObject *object,
guint param_id,
const GValue *value,
- GParamSpec *pspec)
+ GParamSpec *pspec)
{
switch (param_id) {
case PROP_VERBOSITY:
@@ -114,18 +121,59 @@ config_finalize (GObject *object)
static void
config_constructed (GObject *object)
{
+ GSettings *settings;
+
(G_OBJECT_CLASS (tracker_config_parent_class)->constructed) (object);
- g_settings_delay (G_SETTINGS (object));
+ settings = G_SETTINGS (object);
+
+ if (G_LIKELY (!g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
+ g_settings_delay (settings);
+ }
+
+ g_settings_bind (settings, "verbosity", object, "verbosity", G_SETTINGS_BIND_GET | G_SETTINGS_BIND_GET_NO_CHANGES);
}
TrackerConfig *
tracker_config_new (void)
{
- return g_object_new (TRACKER_TYPE_CONFIG,
- "schema-id", "org.freedesktop.Tracker.Writeback",
- "path", "/org/freedesktop/tracker/writeback/",
- NULL);
+ TrackerConfig *config = NULL;
+
+ /* FIXME: should we unset GSETTINGS_BACKEND env var? */
+
+ if (G_UNLIKELY (g_getenv ("TRACKER_USE_CONFIG_FILES"))) {
+ GSettingsBackend *backend;
+ gchar *filename, *basename;
+ gboolean need_to_save;
+
+ basename = g_strdup_printf ("%s.cfg", g_get_prgname ());
+ filename = g_build_filename (g_get_user_config_dir (), "tracker", basename, NULL);
+ g_free (basename);
+
+ need_to_save = g_file_test (filename, G_FILE_TEST_EXISTS) == FALSE;
+
+ backend = g_keyfile_settings_backend_new (filename, CONFIG_PATH, "General");
+ g_info ("Using config file '%s'", filename);
+ g_free (filename);
+
+ config = g_object_new (TRACKER_TYPE_CONFIG,
+ "backend", backend,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
+ g_object_unref (backend);
+
+ if (need_to_save) {
+ g_info (" Config file does not exist, using default values...");
+ }
+ } else {
+ config = g_object_new (TRACKER_TYPE_CONFIG,
+ "schema-id", CONFIG_SCHEMA,
+ "path", CONFIG_PATH,
+ NULL);
+ }
+
+ return config;
}
gint