Merge lp:~unity-team/unity/phablet.rename-unity8 into lp:unity/phablet

Proposed by Michał Sawicz
Status: Merged
Approved by: Sergio Schvezov
Approved revision: no longer in the source branch.
Merged at revision: 736
Proposed branch: lp:~unity-team/unity/phablet.rename-unity8
Merge into: lp:unity/phablet
Diff against target: 339 lines (+83/-53)
15 files modified
CMakeLists.txt (+4/-4)
build (+1/-1)
debian/changelog (+6/-0)
debian/control (+31/-7)
debian/unity8-autopilot.install (+1/-1)
debian/unity8-fake-env.install (+1/-1)
debian/unity8.install (+19/-19)
run (+1/-1)
run_on_device (+2/-2)
runtests.sh (+1/-1)
tests/autopilot/unity8/__init__.py (+1/-1)
tests/autopilot/unity8/tests/__init__.py (+8/-8)
tests/autopilot/unity8/tests/testbigscreen.py (+1/-1)
tests/autopilot/unity8/tests/testcases.py (+2/-2)
unity8.desktop.in (+4/-4)
To merge this branch: bzr merge lp:~unity-team/unity/phablet.rename-unity8
Reviewer Review Type Date Requested Status
Sergio Schvezov Approve
PS Jenkins bot (community) continuous-integration Approve
Didier Roche-Tolomelli packaging Pending
Review via email: mp+167268@code.launchpad.net

Commit message

rename qml-phone-shell to unity8

Description of the change

Since the decision was made for the QML shell to be just "Unity 8", let's rename it!

To post a comment you must log in.
Revision history for this message
Michał Sawicz (saviq) wrote :

I'm tracking down other places where we need to do a rename. Please don't top-approve until then.

Revision history for this message
Michał Sawicz (saviq) wrote :
Revision history for this message
Michael Zanetti (mzanetti) wrote :

Not sure it is a good idea to name the binary unity8. Renaming the binary on ever major release seems useless additional work to me.

I'm also not a big fan of the package having the 8 in the name, but that seems to be common practice so I would be ok with that.

Bad thing is, I don't have a better idea as we obviously can't conflict with current unity. Maybe something like unity-convergence? Sucks too in some ways...

Anyways, I think it might make sense to think a bit more about the binary name...

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

W dniu 04.06.2013 16:09, Michael Zanetti pisze:
> Not sure it is a good idea to name the binary unity8. Renaming the binary on ever major release seems useless additional work to me.
>
> I'm also not a big fan of the package having the 8 in the name, but that seems to be common practice so I would be ok with that.

This is all just a temporary solution, only there to make sure that we
can install both Unity v7 and Unity v8 in parallel. As soon as Unity v8
becomes the default, the package and binary will be renamed to just
"unity" with version 8.

> Bad thing is, I don't have a better idea as we obviously can't conflict with current unity. Maybe something like unity-convergence? Sucks too in some ways...
>
> Anyways, I think it might make sense to think a bit more about the binary name...

I don't think there's value in finding yet another temporary name. And
the decision has been made that this is Unity v8, so why not name it thus?
--
Michał Sawicz <email address hidden>
Canonical Services Ltd.

Revision history for this message
Sergio Schvezov (sergiusens) wrote :

> Related MRs:
>
> lp:~saviq/session-manager/rename-unity8
> lp:~saviq/touch-preview-images/rename-unity8

Not here, this is for raring... I'll create a new config that handles this name change

Revision history for this message
Sergio Schvezov (sergiusens) wrote :
Revision history for this message
Michał Sawicz (saviq) wrote :

Just a follow-up - we decided that unity8, although not a great name for a binary, will make sure no other name "sticks" to the new Unity - which is just a new version - version 8.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Sergio Schvezov (sergiusens) wrote :

Good, works; will need some more extensive testing, but approving for now to get more people in to it

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2013-06-04 08:58:32 +0000
3+++ CMakeLists.txt 2013-06-05 16:32:28 +0000
4@@ -5,7 +5,7 @@
5 set(CMAKE_INSTALL_PREFIX ${CMAKE_BINARY_DIR}/install CACHE PATH "" FORCE)
6 endif()
7
8-project(qml-phone-shell C CXX)
9+project(unity8 C CXX)
10
11 if(${PROJECT_BINARY_DIR} STREQUAL ${PROJECT_SOURCE_DIR})
12 message(FATAL_ERROR "In-tree build attempt detected, aborting. Set your build dir outside your source dir, delete CMakeCache.txt from source root and try again.")
13@@ -56,7 +56,7 @@
14 # Standard install paths
15 include(GNUInstallDirs)
16
17-set(SHELL_APP_DIR ${CMAKE_INSTALL_DATADIR}/qml-phone-shell)
18+set(SHELL_APP_DIR ${CMAKE_INSTALL_DATADIR}/unity8)
19
20 configure_file(paths.h.in ${CMAKE_CURRENT_BINARY_DIR}/paths.h @ONLY)
21
22@@ -90,11 +90,11 @@
23
24 # Autopilot tests
25 include(autopilot)
26-declare_autopilot_test(qml_phone_shell ${CMAKE_SOURCE_DIR}/tests/autopilot/)
27+declare_autopilot_test(unity8 ${CMAKE_SOURCE_DIR}/tests/autopilot/)
28
29 add_subdirectory(tests)
30
31-set(SHELL_APP qml-phone-shell)
32+set(SHELL_APP unity8)
33
34 set(shell_app_HDRS
35 )
36
37=== modified file 'build'
38--- build 2013-06-05 12:54:44 +0000
39+++ build 2013-06-05 16:32:28 +0000
40@@ -1,6 +1,6 @@
41 #!/bin/sh
42
43-PACKAGE=qml-phone-shell
44+PACKAGE=unity8
45 UNITY_BUILD_DIR=$PWD/../unity_build/build
46 SETUP=false
47 CLEAN=false
48
49=== modified file 'debian/changelog'
50--- debian/changelog 2013-05-31 22:31:27 +0000
51+++ debian/changelog 2013-06-05 16:32:28 +0000
52@@ -1,3 +1,9 @@
53+unity8 (7.80.0) saucy; urgency=low
54+
55+ * Rename to unity8.
56+
57+ -- Michał Sawicz <michal.sawicz@canonical.com> Tue, 04 Jun 2013 14:45:29 +0200
58+
59 qml-phone-shell (1.80) raring; urgency=low
60
61 * Focus/unfocus apps on lock/unlock to make sure keyboard is hidden.
62
63=== modified file 'debian/control'
64--- debian/control 2013-06-04 08:58:32 +0000
65+++ debian/control 2013-06-05 16:32:28 +0000
66@@ -1,4 +1,4 @@
67-Source: qml-phone-shell
68+Source: unity8
69 Section: x11
70 Priority: extra
71 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
72@@ -33,7 +33,9 @@
73 Vcs-Bzr: https://code.launchpad.net/~unity-team/unity/phablet
74 Vcs-Browser: https://bazaar.launchpad.net/~unity-team/unity/phablet/files
75
76-Package: qml-phone-shell
77+Package: unity8
78+Replaces: qml-phone-shell (<< 7.80~)
79+Breaks: qml-phone-shell (<< 7.80~)
80 Architecture: amd64 armhf i386
81 Depends: indicators-client (>= 0.6.1),
82 qtdeclarative5-dee-plugin,
83@@ -50,22 +52,44 @@
84 Description: Ubuntu QML shell
85 The Ubuntu QML shell is the primary user interface for Ubuntu devices.
86
87-Package: qml-phone-shell-fake-env
88+Package: qml-phone-shell
89+Depends: unity8, ${misc:Depends}
90+Architecture: all
91+Description: transitional dummy package
92+ This is a transitional dummy package. It can safely be removed.
93+
94+Package: unity8-fake-env
95+Replaces: qml-phone-shell-fake-env (<< 7.80~)
96+Breaks: qml-phone-shell-fake-env (<< 7.80~)
97 Architecture: amd64 armhf i386
98 Depends: ${misc:Depends}, ${shlibs:Depends}
99 Description: Fake environment for running Ubuntu QML shell
100 Provides fake implementations of some QML modules used by Ubuntu QML shell
101 (e.g Ubuntu.Application) so that you can run it in a sandboxed environment.
102
103-Package: qml-phone-shell-autopilot
104+Package: qml-phone-shell-fake-env
105+Depends: unity8-fake-env, ${misc:Depends}
106+Architecture: all
107+Description: transitional dummy package
108+ This is a transitional dummy package. It can safely be removed.
109+
110+Package: unity8-autopilot
111+Replaces: qml-phone-shell-autopilot (<< 7.80~)
112+Breaks: qml-phone-shell-autopilot (<< 7.80~)
113 Architecture: all
114 Depends: libautopilot-qt,
115 libqt5test5,
116 libqt5widgets5,
117 python-evdev,
118- qml-phone-shell (= ${source:Version}),
119- qml-phone-shell-fake-env (= ${source:Version}),
120+ unity8 (= ${source:Version}),
121+ unity8-fake-env (= ${source:Version}),
122 ${misc:Depends},
123 ${shlibs:Depends},
124 Description: Test package for the shell
125- Autopilot tests for the qml-phone-shell package
126+ Autopilot tests for the unity8 package
127+
128+Package: qml-phone-shell-autopilot
129+Depends: unity8-autopilot, ${misc:Depends}
130+Architecture: all
131+Description: transitional dummy package
132+ This is a transitional dummy package. It can safely be removed.
133
134=== renamed file 'debian/qml-phone-shell-autopilot.install' => 'debian/unity8-autopilot.install'
135--- debian/qml-phone-shell-autopilot.install 2013-04-19 13:37:04 +0000
136+++ debian/unity8-autopilot.install 2013-06-05 16:32:28 +0000
137@@ -1,1 +1,1 @@
138-tests/autopilot/qml_phone_shell/* usr/lib/python2.7/dist-packages/qml_phone_shell/
139+tests/autopilot/unity8/* usr/lib/python2.7/dist-packages/unity8/
140
141=== renamed file 'debian/qml-phone-shell-fake-env.install' => 'debian/unity8-fake-env.install'
142--- debian/qml-phone-shell-fake-env.install 2013-04-18 18:01:03 +0000
143+++ debian/unity8-fake-env.install 2013-06-05 16:32:28 +0000
144@@ -1,1 +1,1 @@
145-/usr/share/qml-phone-shell/plugins/mocks/*
146+/usr/share/unity8/plugins/mocks/*
147
148=== renamed file 'debian/qml-phone-shell.install' => 'debian/unity8.install'
149--- debian/qml-phone-shell.install 2013-05-27 07:00:02 +0000
150+++ debian/unity8.install 2013-06-05 16:32:28 +0000
151@@ -1,20 +1,20 @@
152-/usr/bin/qml-phone-shell
153-/usr/share/applications/qml-phone-shell.desktop
154+/usr/bin/unity8
155+/usr/share/applications/unity8.desktop
156 /usr/share/locale/*
157-/usr/share/qml-phone-shell/Applications/*
158-/usr/share/qml-phone-shell/Bottombar/*
159-/usr/share/qml-phone-shell/Components/*
160-/usr/share/qml-phone-shell/Dash/*
161-/usr/share/qml-phone-shell/Greeter/*
162-/usr/share/qml-phone-shell/Hud/*
163-/usr/share/qml-phone-shell/Launcher/*
164-/usr/share/qml-phone-shell/Notifications/*
165-/usr/share/qml-phone-shell/Panel/*
166-/usr/share/qml-phone-shell/Shell.qml
167-/usr/share/qml-phone-shell/SideStage/*
168-/usr/share/qml-phone-shell/graphics/*
169-/usr/share/qml-phone-shell/plugins/HudClient/*
170-/usr/share/qml-phone-shell/plugins/LightDM/*
171-/usr/share/qml-phone-shell/plugins/Ubuntu/*
172-/usr/share/qml-phone-shell/plugins/Unity/*
173-/usr/share/qml-phone-shell/plugins/Utils/*
174+/usr/share/unity8/Applications/*
175+/usr/share/unity8/Bottombar/*
176+/usr/share/unity8/Components/*
177+/usr/share/unity8/Dash/*
178+/usr/share/unity8/Greeter/*
179+/usr/share/unity8/Hud/*
180+/usr/share/unity8/Launcher/*
181+/usr/share/unity8/Notifications/*
182+/usr/share/unity8/Panel/*
183+/usr/share/unity8/Shell.qml
184+/usr/share/unity8/SideStage/*
185+/usr/share/unity8/graphics/*
186+/usr/share/unity8/plugins/HudClient/*
187+/usr/share/unity8/plugins/LightDM/*
188+/usr/share/unity8/plugins/Ubuntu/*
189+/usr/share/unity8/plugins/Unity/*
190+/usr/share/unity8/plugins/Utils/*
191
192=== modified file 'run'
193--- run 2013-06-03 15:33:42 +0000
194+++ run 2013-06-05 16:32:28 +0000
195@@ -1,6 +1,6 @@
196 #!/bin/sh
197
198-QML_PHONE_SHELL_PATH=./builddir/qml-phone-shell
199+QML_PHONE_SHELL_PATH=./builddir/unity8
200 export LD_LIBRARY_PATH=$PWD/../unity_build/build/lib
201 GDB=false
202 FAKE=false
203
204=== modified file 'run_on_device'
205--- run_on_device 2013-05-29 08:21:31 +0000
206+++ run_on_device 2013-06-05 16:32:28 +0000
207@@ -3,8 +3,8 @@
208 USER=phablet
209 USER_ID=32011
210 PASSWORD=phablet
211-PACKAGE=qml-phone-shell
212-BINARY=qml-phone-shell
213+PACKAGE=unity8
214+BINARY=unity8
215 TARGET_IP=${TARGET_IP-127.0.0.1}
216 TARGET_SSH_PORT=${TARGET_SSH_PORT-2222}
217 TARGET_DEBUG_PORT=3768
218
219=== modified file 'runtests.sh'
220--- runtests.sh 2013-04-23 19:50:21 +0000
221+++ runtests.sh 2013-06-05 16:32:28 +0000
222@@ -105,7 +105,7 @@
223 echo " <source>`pwd`</source>" >> $coveragefile
224 echo " </sources>" >> $coveragefile
225 echo " <packages>" >> $coveragefile
226-echo " <package branch-rate=\"0.0\" complexity=\"0.0\" line-rate=\"$linespercentagef\" name=\"qml-phone-shell\">" >> $coveragefile
227+echo " <package branch-rate=\"0.0\" complexity=\"0.0\" line-rate=\"$linespercentagef\" name=\"unity8\">" >> $coveragefile
228 echo " <classes>" >> $coveragefile
229
230 for i in $file_list; do
231
232=== renamed directory 'tests/autopilot/qml_phone_shell' => 'tests/autopilot/unity8'
233=== modified file 'tests/autopilot/unity8/__init__.py'
234--- tests/autopilot/qml_phone_shell/__init__.py 2013-02-26 15:45:50 +0000
235+++ tests/autopilot/unity8/__init__.py 2013-06-05 16:32:28 +0000
236@@ -5,4 +5,4 @@
237 # under the terms of the GNU General Public License version 3, as published
238 # by the Free Software Foundation.
239
240-"""qml-phone-shell autopilot tests and emulators - top level package."""
241+"""unity8 autopilot tests and emulators - top level package."""
242
243=== modified file 'tests/autopilot/unity8/tests/__init__.py'
244--- tests/autopilot/qml_phone_shell/tests/__init__.py 2013-06-03 15:32:21 +0000
245+++ tests/autopilot/unity8/tests/__init__.py 2013-06-05 16:32:28 +0000
246@@ -5,7 +5,7 @@
247 # under the terms of the GNU General Public License version 3, as published
248 # by the Free Software Foundation.
249
250-"""qml-phone-shell autopilot tests."""
251+"""unity8 autopilot tests."""
252
253 import os.path
254
255@@ -14,7 +14,7 @@
256 from autopilot.platform import model
257 from testtools.matchers import Equals
258
259-from qml_phone_shell.emulators.main_window import MainWindow
260+from unity8.emulators.main_window import MainWindow
261
262 class FormFactors(object):
263 Phone, Tablet, Desktop = range(3)
264@@ -43,20 +43,20 @@
265 os.environ['QML2_IMPORT_PATH'] = "../../builddir/tests/mocks:../../builddir/plugins"
266 if geometry != "0x0":
267 self.app = self.launch_test_application(
268- "../../builddir/qml-phone-shell", "-geometry", geometry, "-frameless", app_type='qt')
269+ "../../builddir/unity8", "-geometry", geometry, "-frameless", app_type='qt')
270 else:
271 self.app = self.launch_test_application(
272- "../../builddir/qml-phone-shell", "-fullscreen", app_type='qt')
273+ "../../builddir/unity8", "-fullscreen", app_type='qt')
274
275 def launch_test_installed(self, geometry):
276- os.environ['LD_LIBRARY_PATH'] = "/usr/share/qml-phone-shell/plugins/mocks/LightDM/" + self.lightdm_mock
277- os.environ['QML2_IMPORT_PATH'] = "/usr/share/qml-phone-shell/plugins/mocks:/usr/share/qml-phone-shell/plugins"
278+ os.environ['LD_LIBRARY_PATH'] = "/usr/share/unity8/plugins/mocks/LightDM/" + self.lightdm_mock
279+ os.environ['QML2_IMPORT_PATH'] = "/usr/share/unity8/plugins/mocks:/usr/share/unity8/plugins"
280 if model() == 'Desktop' and geometry != "0x0":
281 self.app = self.launch_test_application(
282- "qml-phone-shell", "-geometry", geometry, "-frameless", app_type='qt')
283+ "unity8", "-geometry", geometry, "-frameless", app_type='qt')
284 else:
285 self.app = self.launch_test_application(
286- "qml-phone-shell", "-fullscreen", app_type='qt')
287+ "unity8", "-fullscreen", app_type='qt')
288
289 def skipWrapper(*args, **kwargs):
290 pass
291
292=== modified file 'tests/autopilot/unity8/tests/testbigscreen.py'
293--- tests/autopilot/qml_phone_shell/tests/testbigscreen.py 2013-03-05 12:57:00 +0000
294+++ tests/autopilot/unity8/tests/testbigscreen.py 2013-06-05 16:32:28 +0000
295@@ -10,7 +10,7 @@
296 from testtools.matchers import Equals
297 from autopilot.matchers import Eventually
298
299-from qml_phone_shell.tests import ShellTestCase
300+from unity8.tests import ShellTestCase
301
302 class TestBig(ShellTestCase):
303 def setUp(self):
304
305=== modified file 'tests/autopilot/unity8/tests/testcases.py'
306--- tests/autopilot/qml_phone_shell/tests/testcases.py 2013-05-29 16:22:49 +0000
307+++ tests/autopilot/unity8/tests/testcases.py 2013-06-05 16:32:28 +0000
308@@ -25,8 +25,8 @@
309
310 from __future__ import absolute_import
311
312-from qml_phone_shell.tests import ShellTestCase, FormFactors
313-from qml_phone_shell.tests.helpers import TestShellHelpers
314+from unity8.tests import ShellTestCase, FormFactors
315+from unity8.tests.helpers import TestShellHelpers
316
317 from autopilot.input import Mouse, Touch, Pointer
318 from testtools.matchers import Equals, NotEquals, GreaterThan, MismatchError
319
320=== renamed file 'qml-phone-shell.desktop.in' => 'unity8.desktop.in'
321--- qml-phone-shell.desktop.in 2013-04-04 08:39:35 +0000
322+++ unity8.desktop.in 2013-06-05 16:32:28 +0000
323@@ -1,10 +1,10 @@
324 [Desktop Entry]
325 Type=Application
326-Name=Unity Next
327-GenericName=qml-phone-shell
328+Name=Unity 8
329+GenericName=unity8
330 Comment=The converged Unity shell
331-Exec=@CMAKE_INSTALL_PREFIX@/@CMAKE_INSTALL_BINDIR@/qml-phone-shell
332+Exec=@CMAKE_INSTALL_PREFIX@/@CMAKE_INSTALL_BINDIR@/unity8
333 Terminal=false
334-Icon=qml-phone-shell
335+Icon=unity8
336 NoDisplay=true
337 X-Ubuntu-Touch=true
338
339=== renamed file 'unity.qmlproject' => 'unity8.qmlproject'

Subscribers

People subscribed via source and target branches

to all changes: