Merge lp:~barry/ubuntu/maverick/computer-janitor/bug-665740 into lp:ubuntu/maverick/computer-janitor

Proposed by Barry Warsaw
Status: Merged
Merged at revision: 20
Proposed branch: lp:~barry/ubuntu/maverick/computer-janitor/bug-665740
Merge into: lp:ubuntu/maverick/computer-janitor
Diff against target: 27 lines (+8/-1)
2 files modified
debian/changelog (+7/-0)
debian/control (+1/-1)
To merge this branch: bzr merge lp:~barry/ubuntu/maverick/computer-janitor/bug-665740
Reviewer Review Type Date Requested Status
Colin Watson Approve
Artur Rona (community) Approve
Review via email: mp+46039@code.launchpad.net

Description of the change

Adds explicit dependency on dbus, since depending on python-dbus does not implicitly add the required dependency.

To post a comment you must log in.
Revision history for this message
Artur Rona (ari-tczew) wrote :

Please remove ~ppa0 from debian/changelog.
I'd like to see rather added dbus under python-argparse (>= 1.1).

review: Needs Fixing
22. By Barry Warsaw

Remove ~ppa prefix for test build.

Revision history for this message
Artur Rona (ari-tczew) wrote :

Change target to maverick-proposed in debian/changelog. However, my 2nd suggestion from my 1st comment wasn't applied.

review: Needs Fixing
Revision history for this message
Barry Warsaw (barry) wrote :

Done, but, wouldn't it make more sense for the dbus requirement to appear next to the python-dbus requirement?

23. By Barry Warsaw

maverick-proposed

Revision history for this message
Artur Rona (ari-tczew) wrote :

It doesn't matter. For me looks fine. Now is core-dev turn.

review: Approve
Revision history for this message
Barry Warsaw (barry) wrote :

Thanks for the review Artur!

Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2010-09-20 08:32:14 +0000
3+++ debian/changelog 2011-01-20 00:29:04 +0000
4@@ -1,3 +1,10 @@
5+computer-janitor (2.0.4-0ubuntu1.1) maverick-proposed; urgency=low
6+
7+ * debian/control: Add explicit dependency on dbus since depending on
8+ python-dbus does not yield an implicit dependency on dbus. (LP: #665740)
9+
10+ -- Barry Warsaw <barry@ubuntu.com> Wed, 19 Jan 2011 19:25:57 -0500
11+
12 computer-janitor (2.0.4-0ubuntu1) maverick; urgency=low
13
14 * Fix icon; given by Pavol Klačanský. (LP: #434431)
15
16=== modified file 'debian/control'
17--- debian/control 2010-09-20 08:32:14 +0000
18+++ debian/control 2011-01-20 00:29:04 +0000
19@@ -13,7 +13,7 @@
20 Package: computer-janitor
21 Architecture: all
22 Depends: ${python:Depends}, python-apt (>= 0.7.9),
23- update-manager-core (>= 1:0.98.1), ${misc:Depends}, python-dbus,
24+ update-manager-core (>= 1:0.98.1), ${misc:Depends}, python-dbus, dbus,
25 python-argparse (>= 1.1)
26 Conflicts: system-cleaner
27 Replaces: system-cleaner

Subscribers

People subscribed via source and target branches

to all changes: