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
1=== modified file 'debian/changelog'
2--- debian/changelog 2013-11-01 15:51:17 +0000
3+++ debian/changelog 2013-11-06 12:10:38 +0000
4@@ -1,8 +1,13 @@
5 checkbox (0.17) UNRELEASED; urgency=low
6
7+ [ Brendan Donegan ]
8 * Incremented changelog
9
10- -- Brendan Donegan <brendan.donegan@canonical.com> Fri, 01 Nov 2013 15:29:31 +0000
11+ [ Zygmunt Krynicki ]
12+ * Make plainbox-service a transitional package that depends on
13+ checkbox-ng-service.
14+
15+ -- Zygmunt Krynicki <zygmunt.krynicki@canonical.com> Wed, 06 Nov 2013 09:45:20 +0100
16
17 checkbox (0.16.14) saucy; urgency=low
18
19
20=== renamed file 'debian/plainbox-service.install' => 'debian/checkbox-ng-service.install'
21--- debian/plainbox-service.install 2013-10-23 07:01:08 +0000
22+++ debian/checkbox-ng-service.install 2013-11-06 12:10:38 +0000
23@@ -1,2 +1,2 @@
24-plainbox/contrib/com.canonical.certification.PlainBox1.service usr/share/dbus-1/services/
25+checkbox-ng/contrib/com.canonical.certification.PlainBox1.service usr/share/dbus-1/services/
26
27
28=== modified file 'debian/control'
29--- debian/control 2013-10-29 10:45:12 +0000
30+++ debian/control 2013-11-06 12:10:38 +0000
31@@ -269,10 +269,18 @@
32 CheckBox command line tool
33
34 Package: plainbox-service
35-Architecture: all
36-Depends: plainbox (>= 0.3+bzr2359), python3-gi, python3-dbus
37-Description: PlainBox D-Bus service
38- File(s) needed for auto-starting PlainBox D-Bus service on demand.
39+Priority: extra
40+Section: oldlibs
41+Architecture: all
42+Depends: checkbox-ng-service
43+Description: PlainBox D-Bus service (transitional package)
44+ Transitional package that can be safely removed
45+
46+Package: checkbox-ng-service
47+Architecture: all
48+Depends: checkbox-ng, python3-gi, python3-dbus
49+Description: CheckBox D-Bus service
50+ File(s) needed for auto-starting CheckBox D-Bus service on demand.
51
52 Package: plainbox-autostart-desktop
53 Architecture: all

Subscribers

People subscribed via source and target branches