Merge lp:~zyga/checkbox/checkbox-packaging into lp:~checkbox-dev/checkbox/checkbox-packaging

Proposed by Zygmunt Krynicki
Status: Merged
Approved by: Sylvain Pineau
Approved revision: no longer in the source branch.
Merged at revision: 33
Proposed branch: lp:~zyga/checkbox/checkbox-packaging
Merge into: lp:~checkbox-dev/checkbox/checkbox-packaging
Diff against target: 53 lines (+19/-6)
3 files modified
debian/changelog (+6/-1)
debian/checkbox-ng-service.install (+1/-1)
debian/control (+12/-4)
To merge this branch: bzr merge lp:~zyga/checkbox/checkbox-packaging
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Review via email: mp+194098@code.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

You could make the transition packages deborphan compliant.
It is recommended to adjust its section to oldlibs and its priority to extra in order to ease deborphan's job.

See: http://www.debian.org/doc/manuals/developers-reference/best-pkging-practices.html#bpp-transition

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Thanks, I didn't know about that.

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

Corrected

Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Perfect, It's time to approve both branches now.

review: Approve
lp:~zyga/checkbox/checkbox-packaging updated
33. By Zygmunt Krynicki

"automatic merge by tarmac [r=sylvain-pineau][bug=][author=zkrynicki]"

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2013-11-01 15:51:17 +0000
+++ debian/changelog 2013-11-06 12:10:38 +0000
@@ -1,8 +1,13 @@
1checkbox (0.17) UNRELEASED; urgency=low1checkbox (0.17) UNRELEASED; urgency=low
22
3 [ Brendan Donegan ]
3 * Incremented changelog4 * Incremented changelog
45
5 -- Brendan Donegan <brendan.donegan@canonical.com> Fri, 01 Nov 2013 15:29:31 +00006 [ Zygmunt Krynicki ]
7 * Make plainbox-service a transitional package that depends on
8 checkbox-ng-service.
9
10 -- Zygmunt Krynicki <zygmunt.krynicki@canonical.com> Wed, 06 Nov 2013 09:45:20 +0100
611
7checkbox (0.16.14) saucy; urgency=low12checkbox (0.16.14) saucy; urgency=low
813
914
=== renamed file 'debian/plainbox-service.install' => 'debian/checkbox-ng-service.install'
--- debian/plainbox-service.install 2013-10-23 07:01:08 +0000
+++ debian/checkbox-ng-service.install 2013-11-06 12:10:38 +0000
@@ -1,2 +1,2 @@
1plainbox/contrib/com.canonical.certification.PlainBox1.service usr/share/dbus-1/services/1checkbox-ng/contrib/com.canonical.certification.PlainBox1.service usr/share/dbus-1/services/
22
33
=== modified file 'debian/control'
--- debian/control 2013-10-29 10:45:12 +0000
+++ debian/control 2013-11-06 12:10:38 +0000
@@ -269,10 +269,18 @@
269 CheckBox command line tool269 CheckBox command line tool
270270
271Package: plainbox-service271Package: plainbox-service
272Architecture: all272Priority: extra
273Depends: plainbox (>= 0.3+bzr2359), python3-gi, python3-dbus273Section: oldlibs
274Description: PlainBox D-Bus service274Architecture: all
275 File(s) needed for auto-starting PlainBox D-Bus service on demand.275Depends: checkbox-ng-service
276Description: PlainBox D-Bus service (transitional package)
277 Transitional package that can be safely removed
278
279Package: checkbox-ng-service
280Architecture: all
281Depends: checkbox-ng, python3-gi, python3-dbus
282Description: CheckBox D-Bus service
283 File(s) needed for auto-starting CheckBox D-Bus service on demand.
276284
277Package: plainbox-autostart-desktop285Package: plainbox-autostart-desktop
278Architecture: all286Architecture: all

Subscribers

People subscribed via source and target branches