Merge lp:~townsend/unity8-desktop-session/add-stable-overlay into lp:unity8-desktop-session/xenial

Proposed by Christopher Townsend
Status: Rejected
Rejected by: Christopher Townsend
Proposed branch: lp:~townsend/unity8-desktop-session/add-stable-overlay
Merge into: lp:unity8-desktop-session/xenial
Diff against target: 210 lines (+146/-3)
9 files modified
data/Makefile.am (+1/-1)
data/stable-overlay-ppa.key (+14/-0)
debian/config (+7/-0)
debian/control (+4/-2)
debian/po/POTFILES.in (+1/-0)
debian/po/templates.pot (+49/-0)
debian/postinst (+28/-0)
debian/postrm (+29/-0)
debian/templates (+13/-0)
To merge this branch: bzr merge lp:~townsend/unity8-desktop-session/add-stable-overlay
Reviewer Review Type Date Requested Status
Stephen M. Webb (community) Approve
Review via email: mp+296590@code.launchpad.net

Commit message

Add stable-overlay PPA when installing the package. Updated Standards-Version to 3.9.6 (no changes required).

To post a comment you must log in.
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

Wouldn't it be better to use add-apt-repository to add/remove the ppa and also maybe add a way to purge packages installed from the ppa with ppa-purge?

Revision history for this message
Christopher Townsend (townsend) wrote :

@jibel,

I considered that, but after talking with some others, it is good form to be able to install this (and any Debian package) on a machine without network access. Using the methods you describe, we would absolutely need network access during package install and package purge.

Revision history for this message
Stephen M. Webb (bregma) wrote :

good to go

review: Approve
Revision history for this message
Stephen M. Webb (bregma) wrote :

Changes acceptable.

review: Approve
85. By Christopher Townsend

Address issues based on RAOF's feedback.

Revision history for this message
Christopher Townsend (townsend) wrote :

Rejecting this as the Technical Board rejected this for an SRU.

Unmerged revisions

85. By Christopher Townsend

Address issues based on RAOF's feedback.

84. By Christopher Townsend

Add debconf interactions for asking the user if they indeed want to enable the Stable Overlay PPA.

83. By Christopher Townsend

Add stable-overlay PPA when installing the package.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/Makefile.am'
2--- data/Makefile.am 2014-09-29 20:46:52 +0000
3+++ data/Makefile.am 2016-06-10 18:29:08 +0000
4@@ -21,7 +21,7 @@
5 lightdm_sessiondir = ${lightdmdir}/sessions
6
7 dist_pkgdata_DATA = \
8- sensors.txt
9+ sensors.txt stable-overlay-ppa.key
10
11 dist_icon_DATA = \
12 custom_unity8-mir_badge.png
13
14=== added file 'data/stable-overlay-ppa.key'
15--- data/stable-overlay-ppa.key 1970-01-01 00:00:00 +0000
16+++ data/stable-overlay-ppa.key 2016-06-10 18:29:08 +0000
17@@ -0,0 +1,14 @@
18+-----BEGIN PGP PUBLIC KEY BLOCK-----
19+Version: SKS 1.1.5
20+Comment: Hostname: keyserver.ubuntu.com
21+
22+mI0EUs00cgEEAJJqaPue5gzQiLB1krT9slYbqVW/bSBpW9+qX8gFI44IVM/Bo3yh9BPAs1RA
23+zja96N0FS6SNlew4JYfk7MBT2sFDGpm3bTKt9Go7muO0JkvKv0vYgrrwqORlWK3SfsYa6Eps
24+CdVzZPAKvGzc8I0XywVgcJhM5okx+3J2naBaSp9NABEBAAG0K0xhdW5jaHBhZCBQUEEgZm9y
25+IENJIFRyYWluIFBQQSBTZXJ2aWNlIFRlYW2IuAQTAQIAIgUCUs00cgIbAwYLCQgHAwIGFQgC
26+CQoLBBYCAwECHgECF4AACgkQhVBBKOzxIEy62gP/T2h98ongV+RXekM1DpgTNoH0PBHrZVj4
27+zfrvrYKZOaxRmJ6TWtzG8tFIuB4gPjaFeenJBhCFaZ9UncFQemS9jztQ/pA049L1N7Tijd8/
28+BKD7gc7tM07+Fq+Q6DT7VuUFiVlfZUwWYzk5UXEk6ctluoIRpnRWUHmh6NssuAgd1Nk=
29+=yfBz
30+-----END PGP PUBLIC KEY BLOCK-----
31+
32
33=== added file 'debian/config'
34--- debian/config 1970-01-01 00:00:00 +0000
35+++ debian/config 2016-06-10 18:29:08 +0000
36@@ -0,0 +1,7 @@
37+#!/bin/sh
38+
39+set -e
40+. /usr/share/debconf/confmodule
41+
42+db_input critical unity8-desktop-session-mir/add-stable-overlay || true
43+db_go
44
45=== modified file 'debian/control'
46--- debian/control 2015-06-09 12:42:50 +0000
47+++ debian/control 2016-06-10 18:29:08 +0000
48@@ -4,12 +4,14 @@
49 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
50 Build-Depends: debhelper (>= 9),
51 dh-autoreconf
52-Standards-Version: 3.9.5
53+Standards-Version: 3.9.6
54
55 Package: unity8-desktop-session-mir
56 Architecture: all
57 Conflicts: ubuntu-desktop-mir
58-Depends: qtubuntu-desktop,
59+Depends: apt,
60+ debconf,
61+ qtubuntu-desktop,
62 qtmir-desktop,
63 unity-system-compositor,
64 unity8,
65
66=== added directory 'debian/po'
67=== added file 'debian/po/POTFILES.in'
68--- debian/po/POTFILES.in 1970-01-01 00:00:00 +0000
69+++ debian/po/POTFILES.in 2016-06-10 18:29:08 +0000
70@@ -0,0 +1,1 @@
71+[type: gettext/rfc822deb] templates
72
73=== added file 'debian/po/templates.pot'
74--- debian/po/templates.pot 1970-01-01 00:00:00 +0000
75+++ debian/po/templates.pot 2016-06-10 18:29:08 +0000
76@@ -0,0 +1,49 @@
77+# SOME DESCRIPTIVE TITLE.
78+# Copyright (C) YEAR THE PACKAGE'S COPYRIGHT HOLDER
79+# This file is distributed under the same license as the unity8-desktop-session package.
80+# FIRST AUTHOR <EMAIL@ADDRESS>, YEAR.
81+#
82+#, fuzzy
83+msgid ""
84+msgstr ""
85+"Project-Id-Version: unity8-desktop-session\n"
86+"Report-Msgid-Bugs-To: unity8-desktop-session@packages.debian.org\n"
87+"POT-Creation-Date: 2016-06-08 16:31-0400\n"
88+"PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
89+"Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
90+"Language-Team: LANGUAGE <LL@li.org>\n"
91+"Language: \n"
92+"MIME-Version: 1.0\n"
93+"Content-Type: text/plain; charset=CHARSET\n"
94+"Content-Transfer-Encoding: 8bit\n"
95+
96+#. Type: boolean
97+#. Description
98+#: ../templates:1001
99+msgid "Would you like to install the Stable Overlay PPA?"
100+msgstr ""
101+
102+#. Type: boolean
103+#. Description
104+#: ../templates:1001
105+msgid ""
106+"The Stable Overlay PPA enables installing the latest packages for the Unity8 "
107+"Desktop Session."
108+msgstr ""
109+
110+#. Type: boolean
111+#. Description
112+#: ../templates:1001
113+msgid ""
114+"Selecting this option will enable the Stable Overlay PPA. You will need to "
115+"run 'apt-get update' and 'apt-get dist-upgrade' to upgrade to the packages "
116+"in the Stable Overlay PPA."
117+msgstr ""
118+
119+#. Type: boolean
120+#. Description
121+#: ../templates:1001
122+msgid ""
123+"If you don not select this option, use 'dpkg-reconfigure unity8-desktop-"
124+"session-mir' to add it later."
125+msgstr ""
126
127=== added file 'debian/postinst'
128--- debian/postinst 1970-01-01 00:00:00 +0000
129+++ debian/postinst 2016-06-10 18:29:08 +0000
130@@ -0,0 +1,28 @@
131+#!/bin/sh
132+
133+set -e
134+
135+. /usr/share/debconf/confmodule
136+
137+stable_overlay_file="/etc/apt/sources.list.d/stable-overlay-xenial.list"
138+stable_overlay_ppa="http://ppa.launchpad.net/ci-train-ppa-service/stable-phone-overlay/ubuntu"
139+
140+if ! grep -q $stable_overlay_ppa /etc/apt/sources.list /etc/apt/sources.list.d/* ; then
141+ db_get unity8-desktop-session-mir/add-stable-overlay
142+
143+ if [ "$RET" = "true" ] ; then
144+ if [ ! -e $stable_overlay_ppa ] ; then
145+ echo "deb $stable_overlay_ppa xenial main" > $stable_overlay_file
146+ echo "deb-src $stable_overlay_ppa xenial main" >> $stable_overlay_file
147+
148+ apt-key add /usr/share/unity8-desktop-session/stable-overlay-ppa.key > /dev/null
149+ else
150+ echo "$stable_overlay_ppa already exists."
151+ echo "You will need to enable the Stable Overlay PPA manually."
152+ fi
153+ fi
154+fi
155+
156+#DEBHELPER#
157+
158+exit 0
159
160=== added file 'debian/postrm'
161--- debian/postrm 1970-01-01 00:00:00 +0000
162+++ debian/postrm 2016-06-10 18:29:08 +0000
163@@ -0,0 +1,29 @@
164+#!/bin/sh
165+
166+set -e
167+
168+. /usr/share/debconf/confmodule
169+
170+stable_overlay_file="/etc/apt/sources.list.d/stable-overlay-xenial.list"
171+stable_overlay_file_md5sum="126bc5ee1cb71607257ea431dd587f2f"
172+stable_overlay_public_key="ECF1204C"
173+
174+if [ "$1" = "purge" ] ; then
175+ if [ -e $stable_overlay_file ] ; then
176+ current_md5sum=`md5sum $stable_overlay_file | awk '{print $1}'`
177+
178+ if [ "$current_md5sum" = "$stable_overlay_file_md5sum" ] ; then
179+ echo "Removing Xenial Stable Overlay PPA..."
180+
181+ rm $stable_overlay_file
182+
183+ apt-key remove $stable_overlay_public_key
184+ fi
185+ fi
186+
187+ db_fset unity8-desktop-session-mir/add-stable-overlay seen false
188+fi
189+
190+#DEBHELPER#
191+
192+exit 0
193
194=== added file 'debian/templates'
195--- debian/templates 1970-01-01 00:00:00 +0000
196+++ debian/templates 2016-06-10 18:29:08 +0000
197@@ -0,0 +1,13 @@
198+Template: unity8-desktop-session-mir/add-stable-overlay
199+Type: boolean
200+Default: true
201+_Description: Would you like to install the Stable Overlay PPA?
202+ The Stable Overlay PPA enables installing the latest packages for the
203+ Unity8 Desktop Session.
204+ .
205+ Selecting this option will enable the Stable Overlay PPA. You will need
206+ to run 'apt-get update' and 'apt-get dist-upgrade' to upgrade to the
207+ packages in the Stable Overlay PPA.
208+ .
209+ If you don not select this option, use 'dpkg-reconfigure
210+ unity8-desktop-session-mir' to add it later.

Subscribers

People subscribed via source and target branches