Merge lp:~sil2100/unity-scope-sshsearch/add_dbus_dep into lp:unity-scope-sshsearch

Proposed by Łukasz Zemczak
Status: Rejected
Rejected by: Łukasz Zemczak
Proposed branch: lp:~sil2100/unity-scope-sshsearch/add_dbus_dep
Merge into: lp:unity-scope-sshsearch
Diff against target: 11 lines (+1/-0)
1 file modified
debian/control (+1/-0)
To merge this branch: bzr merge lp:~sil2100/unity-scope-sshsearch/add_dbus_dep
Reviewer Review Type Date Requested Status
Łukasz Zemczak Disapprove
PS Jenkins bot (community) continuous-integration Needs Fixing
Paweł Stołowski (community) Approve
Review via email: mp+164677@code.launchpad.net

Commit message

We're missing the dbus-x11 dependency for the tests, add it now and fix the FTBFS

Description of the change

We have a FTBFS in the PPA since dbus is not being installed, so the tests are failing (missing /var/lib/dbus/machine-id). Probably. Comment?

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Paweł Stołowski (stolowski) wrote :

It seems to me the root cause is the lack of machine-id file; this file is generated by dbus package (on install and on start of dbus upstart job). python-dbus doesn't pull dbus dependency, so I'm not sure if this solves the problem.

review: Abstain
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

I think this would solve the problem as python-dbus implicates installing dbus. The dependency tree is like this: python-dbus depends on libdbus-1-3, libdbus-1-3 recommends dbus (Ubuntu installs by default the recommends). So implicitly dbus should be installed. The thing is, I don't really have the means to check if it really works or not. Or whether we need python-dbus, the whole thing, or just dbus.

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Actually, you're right. Sadly it seems that the builders do not pull-in recommends by default. So let me just add 'dbus'.

review: Needs Fixing
Revision history for this message
Paweł Stołowski (stolowski) wrote :

+1

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Paweł Stołowski (stolowski) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

It seems that this is a bigger issue...

review: Disapprove

Unmerged revisions

13. By Łukasz Zemczak

Actually, dbus is not enough, we require dbus-launch. So dbus-x11 instead

12. By Łukasz Zemczak

Instead of python-dbus, we need to give dbus, as recommends aren't pulled by default in the builders

11. By Łukasz Zemczak

We're missing the python-dbus dependency, add it now and fix the FTBFS

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2013-03-21 20:51:42 +0000
3+++ debian/control 2013-05-20 09:56:28 +0000
4@@ -11,6 +11,7 @@
5 gir1.2-unity-5.0 (>= 6.91),
6 gir1.2-glib-2.0,
7 python-paramiko,
8+ dbus-x11,
9 Standards-Version: 3.9.4
10 Homepage: https://launchpad.net/unity-scope-sshsearch
11 # If you aren't a member of ~submarine but need to upload packaging changes,

Subscribers

People subscribed via source and target branches

to all changes: