URL Dispatcher

Merge lp:~ted/url-dispatcher/lp1608009-no-message into lp:url-dispatcher/16.10

  1. lp1608009-no-message
  2. Merge into trunk.16.10
Proposed by Ted Gould
Status: Merged
Approved by: Marcus Tomlinson
Approved revision: 92
Merged at revision: 94
Proposed branch: lp:~ted/url-dispatcher/lp1608009-no-message
Merge into: lp:url-dispatcher/16.10
Diff against target: 12 lines (+1/-1)
1 file modified
service/dispatcher.c (+1/-1)
To merge this branch: bzr merge lp:~ted/url-dispatcher/lp1608009-no-message
Reviewer Review Type Date Requested Status
Marcus Tomlinson (community) Approve
unity-api-1-bot continuous-integration Needs Fixing
Review via email: mp+302931@code.launchpad.net

Commit message

Not using a message that isn't set properly

Description of the change

Hate it when GIO sets these to bad values sometimes :-/

To post a comment you must log in.
Revision history for this message
unity-api-1-bot (unity-api-1-bot) wrote :

FAILED: Continuous integration, rev:92
https://jenkins.canonical.com/unity-api-1/job/lp-url-dispatcher-ci/11/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build/382/console
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-0-fetch/388/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/unity-api-1/job/lp-url-dispatcher-ci/11/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Marcus Tomlinson (marcustomlinson) wrote :

Ack

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'service/dispatcher.c'
2--- service/dispatcher.c 2015-09-03 21:50:06 +0000
3+++ service/dispatcher.c 2016-08-15 14:02:27 +0000
4@@ -517,7 +517,7 @@
5
6 if (error != NULL) {
7 if (!g_error_matches(error, G_IO_ERROR, G_IO_ERROR_CANCELLED)) {
8- g_error("Unable to connect to D-Bus: %s", error->message);
9+ g_error("Unable to connect to D-Bus");
10 g_main_loop_quit(mainloop);
11 }
12 g_error_free(error);

Subscribers

People subscribed via source and target branches