Merge lp:~ted/unity8-desktop-session/snap-session into lp:unity8-desktop-session

Proposed by Ted Gould
Status: Merged
Approved by: Christopher Townsend
Approved revision: 110
Merged at revision: 104
Proposed branch: lp:~ted/unity8-desktop-session/snap-session
Merge into: lp:unity8-desktop-session
Diff against target: 154 lines (+74/-4)
9 files modified
Makefile.am (+1/-1)
configure.ac (+2/-0)
data/Makefile.am (+4/-2)
data/unity8-snap.desktop (+6/-0)
debian/control (+19/-1)
debian/unity8-session-snap.install (+4/-0)
scripts/Makefile.am (+22/-0)
scripts/unity8-snap-install (+7/-0)
scripts/unity8-snap-install.1 (+9/-0)
To merge this branch: bzr merge lp:~ted/unity8-desktop-session/snap-session
Reviewer Review Type Date Requested Status
Christopher Townsend Approve
Stephen M. Webb (community) Approve
Review via email: mp+305643@code.launchpad.net

Commit message

Add a Unity8 Session that is contained in a snap

Description of the change

There once was a session *snap* *snap*
That didn't have desktop features early enough to make project management happy *snap* *snap*
So they had to come up with other shipping solutions *snap* *snap*
And this MR is part of that *snap* *snap*

To post a comment you must log in.
Revision history for this message
Stephen M. Webb (bregma) wrote :

I'd like to see a couple of changes, see inline comments.

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

Maybe call the script unity8-snap-install to differentiate that it is for installing the U8 snap session.

Also, although not necessary, but I think a small manpage just explaining what the unity8(-snap)-install script is doing would be nice. Plus, that would get lintian off our back for this:)

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

I built the package, added a local apt repository with the 3 debs generated, then tried to install unity8-session-snap and apt wants to remove the following packages:

The following packages will be REMOVED:
  dbus-user-session ubuntu-desktop ubuntu-session unity8-desktop-session

I don't thikn we want the ubuntu-desktop task and ubuntu-session packages removed. This is due to the Conflicts of dbus-user-session. I don't entirely understand why that is a Conflicts...

review: Needs Information
102. By Ted Gould

Catching up to trunk

103. By Ted Gould

Putting on different lines

104. By Ted Gould

Use dash instead of bash

105. By Ted Gould

Changing name to unity8-snap-install

106. By Ted Gould

Adding a man page

Revision history for this message
Ted Gould (ted) wrote :

Updated from comments.

Yes, we don't want it to conflict long term, that's where things are today. We need to fix the snap before we can remove those.

107. By Ted Gould

Don't conflict with user session

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

Quicky inline nitpick. Will try it out soon.

108. By Ted Gould

Removing comma

Revision history for this message
Stephen M. Webb (bregma) :
review: Approve
109. By Ted Gould

New icon for the snap

110. By Ted Gould

Comment out conflicts for now

Revision history for this message
Ted Gould (ted) wrote :

Removed the conflicts until the snap becomes more usable. Changed the icon so you can tell the difference. Both for zesty until the snap becomes a useful solution.

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

Ok, approving and we'll let it all shake out.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Makefile.am'
2--- Makefile.am 2013-11-26 21:07:31 +0000
3+++ Makefile.am 2016-10-25 16:11:22 +0000
4@@ -17,5 +17,5 @@
5
6 AM_MAKEFLAGS = --no-print-directory
7
8-SUBDIRS = data
9+SUBDIRS = data scripts
10 EXTRA_DIST = autogen.sh
11
12=== modified file 'configure.ac'
13--- configure.ac 2016-02-11 15:29:21 +0000
14+++ configure.ac 2016-10-25 16:11:22 +0000
15@@ -25,4 +25,6 @@
16
17 AC_CONFIG_FILES([Makefile])
18 AC_CONFIG_FILES([data/Makefile])
19+AC_CONFIG_FILES([scripts/Makefile])
20+
21 AC_OUTPUT
22
23=== modified file 'data/Makefile.am'
24--- data/Makefile.am 2016-07-14 14:26:54 +0000
25+++ data/Makefile.am 2016-10-25 16:11:22 +0000
26@@ -24,13 +24,15 @@
27 sensors.txt
28
29 dist_icon_DATA = \
30- custom_unity8_badge.png
31+ custom_unity8_badge.png \
32+ custom_unity8-snap_badge.png
33
34 dist_lightdm_conf_DATA = \
35 45-unity8.conf
36
37 dist_lightdm_session_DATA = \
38- unity8.desktop
39+ unity8.desktop \
40+ unity8-snap.desktop
41
42 dist_bin_SCRIPTS = \
43 lightdm-unity8-session
44
45=== added file 'data/custom_unity8-snap_badge.png'
46Binary files data/custom_unity8-snap_badge.png 1970-01-01 00:00:00 +0000 and data/custom_unity8-snap_badge.png 2016-10-25 16:11:22 +0000 differ
47=== added file 'data/unity8-snap.desktop'
48--- data/unity8-snap.desktop 1970-01-01 00:00:00 +0000
49+++ data/unity8-snap.desktop 2016-10-25 16:11:22 +0000
50@@ -0,0 +1,6 @@
51+[Desktop Entry]
52+Name=Unity8
53+Exec=/snap/bin/unity8-session
54+TryExec=/snap/bin/unity8-session
55+Type=Application
56+X-LightDM-Session-Type=mir
57
58=== modified file 'debian/control'
59--- debian/control 2016-09-28 13:56:39 +0000
60+++ debian/control 2016-10-25 16:11:22 +0000
61@@ -10,7 +10,9 @@
62
63 Package: unity8-desktop-session
64 Architecture: all
65-Conflicts: ubuntu-desktop-mir
66+Conflicts: ubuntu-desktop-mir,
67+# unity8-session-snap
68+# Remove conflicts while in development
69 Replaces: unity8-desktop-session-mir (<< 1.0.13)
70 Breaks: unity8-desktop-session-mir (<< 1.0.13)
71 Depends: mir-graphics-drivers-desktop,
72@@ -45,3 +47,19 @@
73 Priority: extra
74 Description: transitional dummy package
75 This is a transitional dummy package. It can safely be removed.
76+
77+Package: unity8-session-snap
78+Architecture: all
79+# Conflicts: unity8-desktop-session
80+# Remove conflicts while in development
81+Depends: lightdm,
82+ mir-graphics-drivers-desktop,
83+ snapd,
84+ unity-system-compositor,
85+ ${misc:Depends}
86+Description: Unity8 session from a Snap
87+ Provides the files to setup LightDM to run a Snap based Unity8
88+ session. It also, as part of it's execution, installs that snap.
89+ .
90+ This package is provided for evaluation and demonstration purposes only.
91+
92
93=== added file 'debian/unity8-session-snap.install'
94--- debian/unity8-session-snap.install 1970-01-01 00:00:00 +0000
95+++ debian/unity8-session-snap.install 2016-10-25 16:11:22 +0000
96@@ -0,0 +1,4 @@
97+usr/share/lightdm/sessions/unity8-snap.desktop
98+usr/share/unity-greeter/custom_unity8-snap_badge.png
99+usr/share/man/man1/unity8-snap-install.*
100+usr/bin/unity8-snap-install
101
102=== added directory 'scripts'
103=== added file 'scripts/Makefile.am'
104--- scripts/Makefile.am 1970-01-01 00:00:00 +0000
105+++ scripts/Makefile.am 2016-10-25 16:11:22 +0000
106@@ -0,0 +1,22 @@
107+# This file is part of Unity8-session, a desktop session configration
108+# package.
109+#
110+# Copyright 2016 Canonical Ltd.
111+#
112+# unity8-desktop-session is free software: you can redistribute it and/or modify
113+# it under the terms of the GNU General Public License version 3, as published by
114+# the Free Software Foundation.
115+#
116+# This program is distributed in the hope that it will be useful, but WITHOUT ANY
117+# WARRANTY; without even the implied warranties of MERCHANTABILITY, SATISFACTORY
118+# QUALITY, or FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public
119+# License for more details.
120+#
121+# You should have received a copy of the GNU General Public License along with
122+# this program. If not, see <http://www.gnu.org/licenses/>.
123+
124+dist_bin_SCRIPTS = \
125+ unity8-snap-install
126+
127+dist_man_MANS = \
128+ unity8-snap-install.1
129
130=== added file 'scripts/unity8-snap-install'
131--- scripts/unity8-snap-install 1970-01-01 00:00:00 +0000
132+++ scripts/unity8-snap-install 2016-10-25 16:11:22 +0000
133@@ -0,0 +1,7 @@
134+#!/bin/sh
135+
136+if [ -f ~/.snap/auth.json ] ; then
137+ snap install --devmode --channel=edge unity8-session
138+else
139+ sudo snap install --devmode --channel=edge unity8-session
140+fi
141
142=== added file 'scripts/unity8-snap-install.1'
143--- scripts/unity8-snap-install.1 1970-01-01 00:00:00 +0000
144+++ scripts/unity8-snap-install.1 2016-10-25 16:11:22 +0000
145@@ -0,0 +1,9 @@
146+.TH unity8-snap-install "1" "Oct 2016" "unity8-snap-install 1.0.13" "User Commands"
147+
148+.SH NAME
149+unity8-snap-install \- install the Unity8 session snap from snap store
150+
151+.SH DESCRIPTION
152+usage: unity8\-snap\-install
153+.PP
154+Install the Unity8 session snap with the proper arguments passed to the snap command

Subscribers

People subscribed via source and target branches