diff options
author | Sam Thursfield <sam@afuera.me.uk> | 2016-06-09 13:47:07 +0100 |
---|---|---|
committer | Sam Thursfield <sam@afuera.me.uk> | 2016-06-09 15:28:38 +0100 |
commit | e5dd641a0b140535d17430ad17590925ccf578b4 (patch) | |
tree | 34acd68f769c99d70d9589e91c1b113105b7f89a | |
parent | b6c1b5cb638051155ec2bed609ff45c0c2bdc67a (diff) | |
download | tracker-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.c | 7 |
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")); } |