Merge lp:~timo-jyrinki/ubuntu-seeds/ubuntu-touch.remove-qt5default into lp:~ubuntu-core-dev/ubuntu-seeds/ubuntu-touch.saucy

[Timo Jyrinki]

Remove qt5-default. (LP: #1183150)

It comes as part of the ubuntu-sdk for developers, but device images
shouldn't contain them by default. Qtchooser in itself is harmless in
terms of dependencies and is also small, and should be left on images.

The main reason for including qt5-default was the use of qmlscene from a
symlink in /usr/bin instead of the full path. With qtbase and qtchooser
changes uploaded to saucy, qtchooser can be used properly without -dev
packages or qt5-default.

Remove qt5-default. (LP: #1183150)

It comes as part of the ubuntu-sdk for developers, but device images
shouldn't contain them by default. Qtchooser in itself is harmless in
terms of dependencies and is also small, and should be left on images.

The main reason for including qt5-default was the use of qmlscene from a
symlink in /usr/bin instead of the full path. With qtbase and qtchooser
changes uploaded to saucy, qtchooser can be used properly without -dev
packages or qt5-default.

[Michael Terry (mterry)]

This seems premature? qmlrunner isn't in saucy yet.

[Timo Jyrinki (timo-jyrinki)]

Michael: A valid point, it's in the SDK Release PPA but no ubuntu-ui-toolkit uploads to saucy have yet been done. The saucy device images are still created with PPA:s enabled, but that should go away at the same time everything is gotten to the archives.

[lp:~timo-jyrinki/ubuntu-seeds/ubuntu-touch.remove-qt5default]

13. By Timo Jyrinki on 2013-06-07

Merge with trunk.

14. By Timo Jyrinki on 2013-06-07

Restore qtchooser, only qt5-default should be removed.

With the newest Qt5 and qtchooser changes uploaded to saucy, qmlrunner
is not going to be used anymore. qt5-default should still be removed
to prevent needless installation of -dev packages. At the same time,
qmlrunner usage is made obsolete by having qtchooser's configuration
now as part of core libraries.

Accepting this and keeping qmlscene using programs working still requires
either setting QT_SELECT=qt5 globally, or by per app basis setting that
or calling qmlscene with 'qmlscene -qt=qt5 file.qml'.

[Timo Jyrinki (timo-jyrinki)]

Subscribed Sergio who has the "Using qmlscene without qt5-default & qtbase5-dev" in his inbox from a week ago. Raring is already frozen, so we only consider saucy.

This should be accepted when either the QT_SELECT=qt5 is set globally or the apps using /usr/bin/qmlscene have been updated accordingly.

[Dimitri John Ledkov (xnox)]

Claiming ubuntu-core-dev review, just to get this item from the sponsorship queue. As far as I can tell this is for PhoneFoundations to decide when to apply, as the images are still built with PPAs enabled as far as I know. Requested PhoneFoundations review instead.

review: Abstain

[Sergio Schvezov (sergiusens)]

I approve of this, but would add
QT_SELECT=qt5
in
/etc/environment

review: Approve

[Dimitri John Ledkov (xnox)]

> I approve of this, but would add
> QT_SELECT=qt5
> in
> /etc/environment

Why would that be needed? seems like a very odd place. Is debian going to use /etc/environment?

[Dimitri John Ledkov (xnox)]

Wouldn't that break all existing qt4 packages in the archive that haven't been migrated to use QT_SELECT?

[Oliver Grawert (ogra)]

On Do, 2013-06-13 at 17:58 +0000, Dmitrijs Ledkovs wrote:
> > I approve of this, but would add
> > QT_SELECT=qt5
> > in
> > /etc/environment
>
> Why would that be needed? seems like a very odd place. Is debian going to use /etc/environment?

i doubt debian will use the ubuntu-touch seeds in the foreseeable
future :)

[Sergio Schvezov (sergiusens)]

> Wouldn't that break all existing qt4 packages in the archive that haven't been
> migrated to use QT_SELECT?

sorry for not being specific, but just in Ubuntu Touch, a proper solution to clear /etc/environment is still pending

[Timo Jyrinki (timo-jyrinki)]

The proper solution would be that applications would call qmlscene with '-qt=qt5' parameter or set the environment variable themselves, but I suggested the global setting as a workaround to make this change transparent.

A later proper solution will be to move to te new upstream 'qml' tool, when released.

[Oliver Grawert (ogra)]

added teh necessary changes to livecd-rootfs ... merged

=== modified file 'touch'
--- touch	2013-06-05 14:05:43 +0000
+++ touch	2013-06-07 09:04:27 +0000
@@ -65,7 +65,6 @@
 
  * hud
  * qtubuntu
- * qt5-default
  * qtchooser
  * qtdeclarative5-ubuntu-ui-toolkit-plugin
  * webbrowser-app