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

Proposed by Michael Terry on 2013-09-13
Status: Rejected
Rejected by: Michael Terry on 2013-10-24
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 on 2013-10-11
Robert Bruce Park (community) Approve on 2013-10-10
Zoltan Balogh 2013-09-16 Pending
Ubuntu Core Development Team 2013-09-13 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.
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.

Robert Bruce Park (robru) wrote :

Laney, no, we need this to go through.

review: Approve
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.

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
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 on 2013-09-13

Drop qtdeclarative5-qtaudioengine-plugin from sdk

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'sdk'
2--- sdk 2013-09-10 05:33:10 +0000
3+++ sdk 2013-09-13 18:47:33 +0000
4@@ -30,7 +30,6 @@
5 * (qtdeclarative5-friends-plugin)
6 * (qtdeclarative5-localstorage-plugin)
7 * (qtdeclarative5-particles-plugin)
8- * (qtdeclarative5-qtaudioengine-plugin)
9 * (qtdeclarative5-qtmultimedia-plugin)
10 * (qtdeclarative5-qtquick2-plugin)
11 * (qtdeclarative5-qtsensors-plugin)

Subscribers

People subscribed via source and target branches