Merge lp:~mterry/ubuntu-seeds/ubuntu-touch.saucy-noaudioengine into lp:~ubuntu-core-dev/ubuntu-seeds/ubuntu-touch.saucy

Proposed by Michael Terry
Status: Rejected
Rejected by: Michael Terry
Proposed branch: lp:~mterry/ubuntu-seeds/ubuntu-touch.saucy-noaudioengine
Merge into: lp:~ubuntu-core-dev/ubuntu-seeds/ubuntu-touch.saucy
Diff against target: 11 lines (+0/-1)
1 file modified
sdk (+0/-1)
To merge this branch: bzr merge lp:~mterry/ubuntu-seeds/ubuntu-touch.saucy-noaudioengine
Reviewer Review Type Date Requested Status
Timo Jyrinki Approve
Robert Bruce Park (community) Approve
Zoltan Balogh Pending
Ubuntu Core Development Team Pending
Review via email: mp+185568@code.launchpad.net

Commit message

Drop qtdeclarative5-qtaudioengine-plugin from ubuntu-sdk.

Description of the change

Drop qtdeclarative5-qtaudioengine-plugin from ubuntu-sdk.

qtdeclarative5-qtaudioengine-plugin is problematic. It's a part of qtmultimedia, which wants to be in main. But the qtaudioengine plugin requires OpenAL, which pulls in all sorts of dependencies and would be a pain to promote to main.

The only consumer of the audioengine plugin is ubuntu-sdk's recommendation. If we dropped this recommendation, we could drop the plugin from the archive and promote qtmultimedia easily.

However, I don't know if we actually really do want the QtAudioEngine Qml plugin to be a part of the SDK. Can I get confirmation from someone that knows?

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

This is showing up on ths spnosor queue, so I'm setting as WIP. Please merge/whatever when you're ready.

Revision history for this message
Robert Bruce Park (robru) wrote :

Laney, no, we need this to go through.

review: Approve
Revision history for this message
Robert Bruce Park (robru) wrote :

http://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/revision/758 is relevant to this, and (as far as I can tell) evidence that this MP should be approved.

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

Qt AudioEngine can be freely at least temporarily dropped from the SDK contents. It's not currently used by applications as far as I know, but it will be probably a wanted item for some of the game developers. I think getting OpenAL into mains should be a 14.04 goal.

review: Approve
Revision history for this message
Michael Terry (mterry) wrote :

OK, based on your approval, I've pushed a version of this into trusty. This branch is obsolete, as it's targeting saucy.

At a later date, if we really want QtAudioEngine, we can revisit the issue.

Unmerged revisions

91. By Michael Terry

Drop qtdeclarative5-qtaudioengine-plugin from sdk

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'sdk'
--- sdk 2013-09-10 05:33:10 +0000
+++ sdk 2013-09-13 18:47:33 +0000
@@ -30,7 +30,6 @@
30 * (qtdeclarative5-friends-plugin)30 * (qtdeclarative5-friends-plugin)
31 * (qtdeclarative5-localstorage-plugin)31 * (qtdeclarative5-localstorage-plugin)
32 * (qtdeclarative5-particles-plugin)32 * (qtdeclarative5-particles-plugin)
33 * (qtdeclarative5-qtaudioengine-plugin)
34 * (qtdeclarative5-qtmultimedia-plugin)33 * (qtdeclarative5-qtmultimedia-plugin)
35 * (qtdeclarative5-qtquick2-plugin)34 * (qtdeclarative5-qtquick2-plugin)
36 * (qtdeclarative5-qtsensors-plugin)35 * (qtdeclarative5-qtsensors-plugin)

Subscribers

People subscribed via source and target branches