trust-store

Merge lp:~thomas-voss/trust-store/fix-1347739 into lp:trust-store

  1. fix-1347739
  2. Merge into trunk
Proposed by Thomas Voß
Status: Merged
Approved by: Thomas Voß
Approved revision: 25
Merged at revision: 26
Proposed branch: lp:~thomas-voss/trust-store/fix-1347739
Merge into: lp:trust-store
Diff against target: 28 lines (+7/-1)
1 file modified
src/core/trust/resolve.cpp (+7/-1)
To merge this branch: bzr merge lp:~thomas-voss/trust-store/fix-1347739
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Alberto Mardegan (community) Approve
Review via email: mp+228253@code.launchpad.net

Commit message

Install executor on internal session bus connection.

Description of the change

Install executor on internal session bus connection.

To post a comment you must log in.
Revision history for this message
Alberto Mardegan (mardy) wrote :

Code looks OK, I'll try building it and using it in a second.

Revision history for this message
Alberto Mardegan (mardy) wrote :

Yes, I confirm that this fixes the issue.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:25
http://jenkins.qa.ubuntu.com/job/trust-store-ci/21/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/trust-store-utopic-amd64-ci/21/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/trust-store-utopic-armhf-ci/21
        deb: http://jenkins.qa.ubuntu.com/job/trust-store-utopic-armhf-ci/21/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/trust-store-utopic-i386-ci/21/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/trust-store-ci/21/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:25
http://jenkins.qa.ubuntu.com/job/trust-store-ci/23/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/trust-store-utopic-amd64-ci/23/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/trust-store-utopic-armhf-ci/23
        deb: http://jenkins.qa.ubuntu.com/job/trust-store-utopic-armhf-ci/23/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/trust-store-utopic-i386-ci/23/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/trust-store-ci/23/rebuild

review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/core/trust/resolve.cpp'
--- src/core/trust/resolve.cpp 2014-07-15 16:09:43 +0000
+++ src/core/trust/resolve.cpp 2014-07-25 08:28:29 +0000
@@ -24,6 +24,8 @@
24#include "codec.h"24#include "codec.h"
25#include "dbus_interface.h"25#include "dbus_interface.h"
2626
27#include <core/dbus/asio/executor.h>
28
27#include <core/dbus/service.h>29#include <core/dbus/service.h>
28#include <core/dbus/stub.h>30#include <core/dbus/stub.h>
2931
@@ -35,10 +37,14 @@
35{37{
36std::shared_ptr<dbus::Bus> session_bus()38std::shared_ptr<dbus::Bus> session_bus()
37{39{
38 return std::shared_ptr<dbus::Bus>40 std::shared_ptr<dbus::Bus> bus
39 {41 {
40 new dbus::Bus(dbus::WellKnownBus::session)42 new dbus::Bus(dbus::WellKnownBus::session)
41 };43 };
44
45 bus->install_executor(dbus::asio::make_executor(bus));
46
47 return bus;
42}48}
4349
44struct Store : public core::trust::Store50struct Store : public core::trust::Store

Subscribers

People subscribed via source and target branches