summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSam Thursfield <sam@afuera.me.uk>2016-06-09 13:47:07 +0100
committerSam Thursfield <sam@afuera.me.uk>2016-06-09 15:28:38 +0100
commite5dd641a0b140535d17430ad17590925ccf578b4 (patch)
tree34acd68f769c99d70d9589e91c1b113105b7f89a
parentb6c1b5cb638051155ec2bed609ff45c0c2bdc67a (diff)
downloadtracker-e5dd641a0b140535d17430ad17590925ccf578b4.tar.gz
tracker-extract: Improve console output for when extraction fails
Old output: tracker_mimetype_info_get_module: assertion 'info != NULL' failed No metadata or extractor modules found to handle this file New output: file:///uri/of/file: No metadata or extractor modules found to handle this file The message is now also printed to stderr rather than stdout, so that stdout contains only the metadata itself in whatever output format the caller chose.
-rw-r--r--src/tracker-extract/tracker-extract.c7
1 files changed, 5 insertions, 2 deletions
diff --git a/src/tracker-extract/tracker-extract.c b/src/tracker-extract/tracker-extract.c
index 04b3291df..a184078a7 100644
--- a/src/tracker-extract/tracker-extract.c
+++ b/src/tracker-extract/tracker-extract.c
@@ -789,7 +789,9 @@ tracker_extract_get_metadata_by_cmdline (TrackerExtract *object,
}
task->mimetype_handlers = tracker_extract_module_manager_get_mimetype_handlers (task->mimetype);
- task->cur_module = tracker_mimetype_info_get_module (task->mimetype_handlers, &task->cur_func, NULL);
+ if (task->mimetype_handlers) {
+ task->cur_module = tracker_mimetype_info_get_module (task->mimetype_handlers, &task->cur_func, NULL);
+ }
while (task->cur_func) {
if (!filter_module (object, task->cur_module) &&
@@ -845,7 +847,8 @@ tracker_extract_get_metadata_by_cmdline (TrackerExtract *object,
}
if (no_data_or_modules) {
- g_print ("%s\n\n",
+ g_printerr ("%s: %s\n",
+ uri,
_("No metadata or extractor modules found to handle this file"));
}