[tracker/writeback] Implemented writeback for MP3's Title field using id3lib
- From: Philip Van Hoof <pvanhoof src gnome org>
- To: svn-commits-list gnome org
- Cc:
- Subject: [tracker/writeback] Implemented writeback for MP3's Title field using id3lib
- Date: Wed, 11 Nov 2009 17:08:34 +0000 (UTC)
commit 3bfa19711901f8424fdd7a7e9b63018f57646249
Author: Philip Van Hoof <philip codeminded be>
Date: Wed Nov 11 18:06:56 2009 +0100
Implemented writeback for MP3's Title field using id3lib
This is a prototype implementation
configure.ac | 51 ++++++++
data/ontologies/30-nie.ontology | 3 +-
data/ontologies/90-tracker.ontology | 7 -
src/tracker-writeback/Makefile.am | 17 ++-
.../tracker-writeback-dispatcher.c | 56 ++++-----
src/tracker-writeback/tracker-writeback-dummy.c | 103 ---------------
src/tracker-writeback/tracker-writeback-mp3.c | 132 ++++++++++++++++++++
src/tracker-writeback/tracker-writeback.c | 4 +-
src/tracker-writeback/tracker-writeback.h | 2 -
9 files changed, 222 insertions(+), 153 deletions(-)
---
diff --git a/configure.ac b/configure.ac
index bc81757..97ac625 100644
--- a/configure.ac
+++ b/configure.ac
@@ -154,6 +154,7 @@ EDS_REQUIRED=2.25.5
# Unlikely version for now, Nepomuk integration isn't finished in streamanalyzer atm
LIBSTREAMANALYZER_REQUIRED=0.7.0
GEE_REQUIRED=0.3
+ID3LIB_REQUIRED=3.8.3
# Library Checks
PKG_CHECK_MODULES(GLIB2, [glib-2.0 >= $GLIB_REQUIRED])
@@ -229,6 +230,52 @@ fi
GLIB_GENMARSHAL=`$PKG_CONFIG glib-2.0 --variable=glib_genmarshal`
AC_SUBST(GLIB_GENMARSHAL)
+####################################################################
+# ID3lib for MP3 writeback support
+####################################################################
+
+# FIXME This should be package based. Unfortunately in several main
+# distros, it is not.
+
+OLD_CFLAGS="$CFLAGS"
+OLD_LIBS="$LIBS"
+CFLAGS=""
+LIBS=""
+
+AC_ARG_ENABLE(id3lib,
+ AS_HELP_STRING([--enable-id3lib],
+ [enable writeback for MP3 data [[default=auto]]]),,
+ [enable_id3lib=auto])
+
+if test "x$enable_id3lib" != "xno" ; then
+ AC_CHECK_HEADER(id3.h,
+ AC_CHECK_LIB(id3, ID3Tag_New))
+
+ have_id3lib=${ac_cv_lib_id3_ID3Tag_New:-no}
+
+ ID3LIB_CFLAGS="$CFLAGS"
+ ID3LIB_LIBS="$LIBS"
+
+ AC_SUBST(ID3LIB_CFLAGS)
+ AC_SUBST(ID3LIB_LIBS)
+
+ if test "x$have_libjpeg" = "xyes"; then
+ AC_DEFINE(HAVE_ID3LIB, [], [Define if we have id3lib])
+ fi
+else
+ have_id3lib="no (disabled)"
+fi
+
+if test "x$enable_id3lib" = "xyes"; then
+ if test "x$have_id3lib" != "xyes"; then
+ AC_MSG_ERROR([Couldn't find id3lib >= $ID3LIB_REQUIRED.])
+ fi
+fi
+
+AM_CONDITIONAL(HAVE_ID3LIB, test "x$have_id3lib" = "xyes")
+
+CFLAGS="$OLD_CFLAGS"
+LIBS="$OLD_LIBS"
####################################################################
# Stream Analyzer
@@ -1557,6 +1604,10 @@ Plugins:
Evolution plugin (data-push): $have_evolution_plugin
KMail plugin (data-push): $enable_kmail_push_module
+Writeback:
+
+ MP3 writeback: $have_id3lib
+
Warning:
You must make sure SQLite is compiled with --enable-threadsafe
diff --git a/data/ontologies/30-nie.ontology b/data/ontologies/30-nie.ontology
index a7b493e..4d77380 100644
--- a/data/ontologies/30-nie.ontology
+++ b/data/ontologies/30-nie.ontology
@@ -29,7 +29,8 @@ nie:title a rdf:Property ;
rdfs:domain nie:InformationElement ;
rdfs:range xsd:string ;
tracker:fulltextIndexed true ;
- tracker:weight 10 .
+ tracker:weight 10 ;
+ tracker:writeback true .
nie:subject a rdf:Property ;
rdfs:label "Subject" ;
diff --git a/data/ontologies/90-tracker.ontology b/data/ontologies/90-tracker.ontology
index 709a9a1..88e9936 100644
--- a/data/ontologies/90-tracker.ontology
+++ b/data/ontologies/90-tracker.ontology
@@ -49,12 +49,5 @@ tracker:writeback a rdf:Property ;
rdfs:domain rdf:Property ;
rdfs:range xsd:boolean .
-# Remove this afterward
-tracker:writebackTestField a rdf:Property ;
- nrl:maxCardinality 1 ;
- tracker:writeback true ;
- rdfs:domain rdfs:Resource ;
- rdfs:range xsd:string .
-
fts: a tracker:Namespace ;
tracker:prefix "fts" .
diff --git a/src/tracker-writeback/Makefile.am b/src/tracker-writeback/Makefile.am
index 9e9d6aa..3c20f99 100644
--- a/src/tracker-writeback/Makefile.am
+++ b/src/tracker-writeback/Makefile.am
@@ -14,15 +14,18 @@ INCLUDES = \
$(GLIB2_CFLAGS) \
$(GCOV_CFLAGS) \
$(GMODULE_CFLAGS) \
- $(DBUS_CFLAGS)
-
-modules_LTLIBRARIES = \
- libwriteback-dummy.la
-
-# dummy
-libwriteback_dummy_la_SOURCES = tracker-writeback-dummy.c
-libwriteback_dummy_la_LDFLAGS = $(module_flags)
-libwriteback_dummy_la_LIBADD = $(GLIB2_LIBS) $(GCOV_LIBS)
+ $(DBUS_CFLAGS) \
+ $(ID3LIB_CFLAGS)
+
+modules_LTLIBRARIES =
+
+# MP3
+if HAVE_ID3LIB
+modules_LTLIBRARIES+=libwriteback-mp3.la
+libwriteback_mp3_la_SOURCES = tracker-writeback-mp3.c
+libwriteback_mp3_la_LDFLAGS = $(module_flags)
+libwriteback_mp3_la_LIBADD = $(GLIB2_LIBS) $(GCOV_LIBS) $(ID3LIB_LIBS)
+endif
#
# Binaries
diff --git a/src/tracker-writeback/tracker-writeback-dispatcher.c b/src/tracker-writeback/tracker-writeback-dispatcher.c
index beeaa4a..2e0dabd 100644
--- a/src/tracker-writeback/tracker-writeback-dispatcher.c
+++ b/src/tracker-writeback/tracker-writeback-dispatcher.c
@@ -39,8 +39,7 @@ typedef struct {
} TrackerWritebackDispatcherPrivate;
typedef struct {
- TrackerWritebackDispatcher *dispatcher;
- GFile *file;
+ TrackerWritebackDispatcher *dispatcher;
} QueryData;
#define TRACKER_WRITEBACK_DISPATCHER_GET_PRIVATE(o) (G_TYPE_INSTANCE_GET_PRIVATE ((o), TRACKER_TYPE_WRITEBACK_DISPATCHER, TrackerWritebackDispatcherPrivate))
@@ -260,33 +259,30 @@ on_sparql_result_received (GPtrArray *result,
gpointer user_data)
{
TrackerWritebackDispatcherPrivate *priv;
- TrackerWritebackModule *module;
- TrackerWriteback *writeback;
- gchar *subject = user_data;
- GHashTableIter iter;
- gpointer key, value;
- QueryData *data;
+ TrackerWritebackModule *module;
+ TrackerWriteback *writeback;
+ GHashTableIter iter;
+ gpointer key, value;
+ QueryData *data;
- data = user_data;
- priv = TRACKER_WRITEBACK_DISPATCHER_GET_PRIVATE (data->dispatcher);
+ data = user_data;
+ priv = TRACKER_WRITEBACK_DISPATCHER_GET_PRIVATE (data->dispatcher);
- /* FIXME: Lookup module by mimetype */
- g_hash_table_iter_init (&iter, priv->modules);
+ /* FIXME: Lookup module by mimetype */
+ g_hash_table_iter_init (&iter, priv->modules);
- if (!g_hash_table_iter_next (&iter, &key, &value)) {
- return;
- }
+ if (!g_hash_table_iter_next (&iter, &key, &value)) {
+ return;
+ }
- module = value;
- writeback = tracker_writeback_module_create (module);
+ module = value;
+ writeback = tracker_writeback_module_create (module);
- tracker_writeback_update_metadata (writeback, data->file, result);
+ tracker_writeback_update_metadata (writeback, result);
- g_object_unref (writeback);
+ g_object_unref (writeback);
- g_object_unref (data->file);
- g_slice_free (QueryData, data);
- g_free (subject);
+ g_slice_free (QueryData, data);
}
static void
@@ -295,7 +291,7 @@ on_writeback_cb (DBusGProxy *proxy,
TrackerWritebackDispatcher *object)
{
TrackerWritebackDispatcherPrivate *priv;
- QueryData *data;
+ QueryData *data;
guint n;
priv = TRACKER_WRITEBACK_DISPATCHER_GET_PRIVATE (object);
@@ -303,20 +299,20 @@ on_writeback_cb (DBusGProxy *proxy,
for (n = 0; subjects[n] != NULL; n++) {
gchar *query;
-
- query = g_strdup_printf ("SELECT ?predicate ?object { "
- "<%s> ?predicate ?object . "
+ query = g_strdup_printf ("SELECT ?url ?predicate ?object { "
+ "<%s> ?predicate ?object ; "
+ "nie:isStoredAs ?url . "
"?predicate tracker:writeback true "
"}", subjects[n]);
- data = g_slice_new (QueryData);
- data->dispatcher = object;
- data->file = g_file_new_for_uri (subjects[n]);
+ data = g_slice_new (QueryData);
+ data->dispatcher = object;
tracker_resources_sparql_query_async (priv->client,
query,
on_sparql_result_received,
- data);
+ data);
+
g_free (query);
}
}
diff --git a/src/tracker-writeback/tracker-writeback-mp3.c b/src/tracker-writeback/tracker-writeback-mp3.c
new file mode 100644
index 0000000..6bb78e7
--- /dev/null
+++ b/src/tracker-writeback/tracker-writeback-mp3.c
@@ -0,0 +1,132 @@
+/* -*- Mode: C; tab-width: 8; indent-tabs-mode: t; c-basic-offset: 8 -*- */
+/*
+ * Copyright (C) 2008, Nokia
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public
+ * License along with this library; if not, write to the
+ * Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
+ * Boston, MA 02110-1301, USA.
+ *
+ * Authors: Philip Van Hoof <philip codeminded be>
+ */
+
+#include <id3.h>
+
+#include <libtracker-common/tracker-ontology.h>
+
+#include "tracker-writeback.h"
+
+#define TRACKER_TYPE_WRITEBACK_MP3 (tracker_writeback_mp3_get_type ())
+
+typedef struct TrackerWritebackMP3 TrackerWritebackMP3;
+typedef struct TrackerWritebackMP3Class TrackerWritebackMP3Class;
+
+struct TrackerWritebackMP3 {
+ TrackerWriteback parent_instance;
+};
+
+struct TrackerWritebackMP3Class {
+ TrackerWritebackClass parent_class;
+};
+
+static GType tracker_writeback_mp3_get_type (void) G_GNUC_CONST;
+static gboolean tracker_writeback_mp3_update_metadata (TrackerWriteback *writeback,
+ GPtrArray *values);
+
+G_DEFINE_DYNAMIC_TYPE (TrackerWritebackMP3, tracker_writeback_mp3, TRACKER_TYPE_WRITEBACK);
+
+static void
+tracker_writeback_mp3_class_init (TrackerWritebackMP3Class *klass)
+{
+ TrackerWritebackClass *writeback_class = TRACKER_WRITEBACK_CLASS (klass);
+
+ writeback_class->update_metadata = tracker_writeback_mp3_update_metadata;
+}
+
+static void
+tracker_writeback_mp3_class_finalize (TrackerWritebackMP3Class *klass)
+{
+}
+
+static void
+tracker_writeback_mp3_init (TrackerWritebackMP3 *mp3)
+{
+}
+
+static gboolean
+tracker_writeback_mp3_update_metadata (TrackerWriteback *writeback,
+ GPtrArray *values)
+{
+ guint n;
+ const gchar *uri = NULL;
+ GFile *file = NULL;
+ gchar *path = NULL;
+
+ for (n = 0; n < values->len; n++) {
+ const GStrv row = g_ptr_array_index (values, n);
+
+ if (uri == NULL) {
+ uri = row[0];
+ file = g_file_new_for_uri (uri);
+ path = g_file_get_path (file);
+ }
+
+ if (g_strcmp0 (row[1], TRACKER_NIE_PREFIX "title") == 0) {
+ ID3Tag *tag = ID3Tag_New ();
+ ID3Frame *frame;
+
+ ID3Tag_Link (tag, path);
+
+ frame = ID3Tag_FindFrameWithID (tag, ID3FID_TITLE);
+ if (frame) {
+ ID3Field *field;
+ field = ID3Frame_GetField (frame, ID3FN_TEXT);
+ ID3Field_SetASCII (field, row[2]);
+ } else {
+ ID3Field *field;
+ frame = ID3Frame_NewID (ID3FID_TITLE);
+ field = ID3Frame_GetField (frame, ID3FN_TEXT);
+ ID3Field_SetASCII (field, row[2]);
+ ID3Tag_AddFrame (tag, frame);
+ }
+
+ ID3Tag_Update (tag);
+ // ID3Frame_Delete (frame);
+ ID3Tag_Delete (tag);
+ }
+ }
+
+ if (path)
+ g_free (path);
+
+ if (file)
+ g_object_unref (file);
+
+ return TRUE;
+}
+
+TrackerWriteback *
+writeback_module_create (GTypeModule *module)
+{
+ tracker_writeback_mp3_register_type (module);
+
+ return g_object_new (TRACKER_TYPE_WRITEBACK_MP3, NULL);
+}
+
+const GStrv
+writeback_module_get_mimetypes (void)
+{
+ static const gchar *mimetypes[] = { "*", NULL };
+
+ return (GStrv) mimetypes;
+}
diff --git a/src/tracker-writeback/tracker-writeback.c b/src/tracker-writeback/tracker-writeback.c
index 09e64a2..0ee37ea 100644
--- a/src/tracker-writeback/tracker-writeback.c
+++ b/src/tracker-writeback/tracker-writeback.c
@@ -36,15 +36,13 @@ tracker_writeback_init (TrackerWriteback *writeback)
gboolean
tracker_writeback_update_metadata (TrackerWriteback *writeback,
- GFile *file,
GPtrArray *values)
{
g_return_val_if_fail (TRACKER_IS_WRITEBACK (writeback), FALSE);
- g_return_val_if_fail (G_IS_FILE (file), FALSE);
g_return_val_if_fail (values != NULL, FALSE);
if (TRACKER_WRITEBACK_GET_CLASS (writeback)->update_metadata) {
- return TRACKER_WRITEBACK_GET_CLASS (writeback)->update_metadata (writeback, file, values);
+ return TRACKER_WRITEBACK_GET_CLASS (writeback)->update_metadata (writeback, values);
}
return FALSE;
diff --git a/src/tracker-writeback/tracker-writeback.h b/src/tracker-writeback/tracker-writeback.h
index 337115c..ab3bd53 100644
--- a/src/tracker-writeback/tracker-writeback.h
+++ b/src/tracker-writeback/tracker-writeback.h
@@ -44,7 +44,6 @@ struct TrackerWritebackClass {
GObjectClass parent_class;
gboolean (* update_metadata) (TrackerWriteback *writeback,
- GFile *file,
GPtrArray *values);
};
@@ -55,7 +54,6 @@ TrackerWriteback * writeback_module_create (GTypeModule *module);
const GStrv writeback_module_get_mimetypes (void);
gboolean tracker_writeback_update_metadata (TrackerWriteback *writeback,
- GFile *file,
GPtrArray *values);
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]
