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
=== modified file 'debian/control'
--- debian/control 2013-05-23 07:43:46 +0000
+++ debian/control 2013-06-12 05:37:29 +0000
@@ -108,20 +108,3 @@
108Description: Test package for Ubuntu UI Toolkit108Description: Test package for Ubuntu UI Toolkit
109 Autopilot tests for the ubuntu-ui-toolkit package109 Autopilot tests for the ubuntu-ui-toolkit package
110110
111Package: qmlrunner
112Architecture: amd64 i386 armhf
113Depends: qmlscene
114Conflicts: qtchooser
115Description: Simple QML Scene symlink provider
116 A simple qmlscene symlink provider. Depend on this package instead of
117 qtchooser/qt5-default/qmlscene in the following way:
118 .
119 Depends: qmlrunner | qtchooser
120 .
121 The conditional dependency allows either having just qmlrunner installed,
122 or having for example full ubuntu-sdk (and qt5-default/qtchooser).
123 .
124 This package will be eventually replaced by upstream 'qml' tool, so
125 it is encouraged to use the '/usr/bin/qml' symlink already which is also
126 provided by this package.
127
128111
=== removed file 'debian/qmlrunner.install'
--- debian/qmlrunner.install 2013-05-08 05:25:53 +0000
+++ debian/qmlrunner.install 1970-01-01 00:00:00 +0000
@@ -1,2 +0,0 @@
1usr/bin/qml
2usr/bin/qmlscene
30
=== modified file 'debian/rules'
--- debian/rules 2013-05-29 04:56:18 +0000
+++ debian/rules 2013-06-12 05:37:29 +0000
@@ -8,15 +8,6 @@
8%:8%:
9 dh $@ --fail-missing9 dh $@ --fail-missing
1010
11override_dh_auto_install:
12 dh_auto_install
13 # Temporary hacks until proper upstream binary installed
14 # Provide both legacy qmlscene and future qml links
15 # -> users may update to use the 'qml' name
16 mkdir -p $(CURDIR)/debian/tmp/usr/bin
17 ln -s /usr/lib/$(DEB_HOST_MULTI_ARCH)/qt5/bin/qmlscene $(CURDIR)/debian/tmp/usr/bin/qml
18 ln -s /usr/lib/$(DEB_HOST_MULTI_ARCH)/qt5/bin/qmlscene $(CURDIR)/debian/tmp/usr/bin/qmlscene
19
20override_dh_install:11override_dh_install:
21 dh_auto_build -- docs12 dh_auto_build -- docs
22 mkdir -p debian/tmp/`qmake -query QT_INSTALL_DOCS`/qch 13 mkdir -p debian/tmp/`qmake -query QT_INSTALL_DOCS`/qch

Subscribers

People subscribed via source and target branches

to status/vote changes: