Merge lp:~timo-jyrinki/ubuntu-ui-toolkit/remove_qmlrunner into lp:ubuntu-ui-toolkit

Proposed by Timo Jyrinki
Status: Merged
Approved by: Florian Boucault
Approved revision: 540
Merged at revision: 604
Proposed branch: lp:~timo-jyrinki/ubuntu-ui-toolkit/remove_qmlrunner
Merge into: lp:ubuntu-ui-toolkit
Diff against target: 51 lines (+0/-28)
3 files modified
debian/control (+0/-17)
debian/qmlrunner.install (+0/-2)
debian/rules (+0/-9)
To merge this branch: bzr merge lp:~timo-jyrinki/ubuntu-ui-toolkit/remove_qmlrunner
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Ubuntu SDK team Pending
Review via email: mp+168851@code.launchpad.net

Commit message

Remove qmlrunner. On saucy, the less complicated way is to just use qmlscene with either environment variable set or with a command line parameter.

Description of the change

Remove qmlrunner. On saucy, the less complicated way if to just use qmlscene with either environment variable set or with a command line parameter.

See eg. https://code.launchpad.net/~timo-jyrinki/ubuntu-seeds/ubuntu-touch.remove-qt5default/+merge/165291 for rationale.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Florian Boucault (fboucault) wrote :

Is anybody using qmlrunner out there? any app, anything?

Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Not that I know of. I also grepped a whole device image for qmlrunner with no results.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2013-05-23 07:43:46 +0000
3+++ debian/control 2013-06-12 05:37:29 +0000
4@@ -108,20 +108,3 @@
5 Description: Test package for Ubuntu UI Toolkit
6 Autopilot tests for the ubuntu-ui-toolkit package
7
8-Package: qmlrunner
9-Architecture: amd64 i386 armhf
10-Depends: qmlscene
11-Conflicts: qtchooser
12-Description: Simple QML Scene symlink provider
13- A simple qmlscene symlink provider. Depend on this package instead of
14- qtchooser/qt5-default/qmlscene in the following way:
15- .
16- Depends: qmlrunner | qtchooser
17- .
18- The conditional dependency allows either having just qmlrunner installed,
19- or having for example full ubuntu-sdk (and qt5-default/qtchooser).
20- .
21- This package will be eventually replaced by upstream 'qml' tool, so
22- it is encouraged to use the '/usr/bin/qml' symlink already which is also
23- provided by this package.
24-
25
26=== removed file 'debian/qmlrunner.install'
27--- debian/qmlrunner.install 2013-05-08 05:25:53 +0000
28+++ debian/qmlrunner.install 1970-01-01 00:00:00 +0000
29@@ -1,2 +0,0 @@
30-usr/bin/qml
31-usr/bin/qmlscene
32
33=== modified file 'debian/rules'
34--- debian/rules 2013-05-29 04:56:18 +0000
35+++ debian/rules 2013-06-12 05:37:29 +0000
36@@ -8,15 +8,6 @@
37 %:
38 dh $@ --fail-missing
39
40-override_dh_auto_install:
41- dh_auto_install
42- # Temporary hacks until proper upstream binary installed
43- # Provide both legacy qmlscene and future qml links
44- # -> users may update to use the 'qml' name
45- mkdir -p $(CURDIR)/debian/tmp/usr/bin
46- ln -s /usr/lib/$(DEB_HOST_MULTI_ARCH)/qt5/bin/qmlscene $(CURDIR)/debian/tmp/usr/bin/qml
47- ln -s /usr/lib/$(DEB_HOST_MULTI_ARCH)/qt5/bin/qmlscene $(CURDIR)/debian/tmp/usr/bin/qmlscene
48-
49 override_dh_install:
50 dh_auto_build -- docs
51 mkdir -p debian/tmp/`qmake -query QT_INSTALL_DOCS`/qch

Subscribers

People subscribed via source and target branches

to status/vote changes: