Merge lp:~unity-team/unity/phablet.rename-unity8 into lp:unity/phablet
- phablet.rename-unity8
- Merge into phablet
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 |
Related bugs: |
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!
Michał Sawicz (saviq) wrote : | # |
Michał Sawicz (saviq) wrote : | # |
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...
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:732
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
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.
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
Sergio Schvezov (sergiusens) wrote : | # |
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.
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:732
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:732
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:733
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
Click here to trigger a rebuild:
http://
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:733
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
Click here to trigger a rebuild:
http://
PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:734
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Sergio Schvezov (sergiusens) wrote : | # |
Good, works; will need some more extensive testing, but approving for now to get more people in to it
Preview Diff
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' |
I'm tracking down other places where we need to do a rename. Please don't top-approve until then.