Merge lp:~cjwatson/webbrowser-app/allow-gles into lp:webbrowser-app

Proposed by Colin Watson
Status: Merged
Approved by: Olivier Tilloy
Approved revision: 1140
Merged at revision: 1141
Proposed branch: lp:~cjwatson/webbrowser-app/allow-gles
Merge into: lp:webbrowser-app
Diff against target: 48 lines (+5/-5)
1 file modified
debian/control (+5/-5)
To merge this branch: bzr merge lp:~cjwatson/webbrowser-app/allow-gles
Reviewer Review Type Date Requested Status
Olivier Tilloy Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+268180@code.launchpad.net

Commit message

Fix versioned dependencies on qtdeclarative5-ubuntu-ui-toolkit-plugin to allow qtdeclarative5-ubuntu-ui-toolkit-plugin-gles.

Description of the change

https://code.launchpad.net/~osomon/webbrowser-app/uitk-1.3/+merge/267544 regressed installability of the ubuntu-touch metapackage on amd64 and i386: a >= versioned dependency may not be satisfied by a virtual package, but on amd64/i386 we rely on being able to use -gles variants of packages. We thus need to explicitly allow qtdeclarative5-ubuntu-ui-toolkit-plugin-gles.

Incidentally, this change from that MP looks somewhat dubious and possibly a mistake:

- qtdeclarative5-ubuntu-ui-extras-browser-plugin (= ${binary:Version}),
+ qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),

... but I've simply fixed that up to allow -gles rather than reverting it, in case it was in fact intentional.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Olivier Tilloy (osomon) wrote :

Thanks for that Colin.

The other change that looked dubious was in fact intentional, I should have explained it in the merge request description. qtdeclarative5-ubuntu-ui-extras-browser-plugin has been deprecated for over a year (and replaced by qtdeclarative5-ubuntu-web-plugin), this dependency was a useless leftover. And the fact that webapp-container didn’t depend on qtdeclarative5-ubuntu-ui-toolkit-plugin was obviously a mistake, although a harmless one since it pulled the dependency indirectly anyway.

review: Approve

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 2015-08-11 19:05:10 +0000
3+++ debian/control 2015-08-16 23:16:32 +0000
4@@ -21,7 +21,7 @@
5 qtbase5-private-dev,
6 qtdeclarative5-dev,
7 qtdeclarative5-private-dev,
8- qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),
9+ qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.3),
10 xvfb,
11 Standards-Version: 3.9.5
12 Homepage: https://launchpad.net/webbrowser-app
13@@ -42,7 +42,7 @@
14 qml-module-qtquick2 (>= 5.4),
15 qml-module-qtquick-dialogs,
16 qml-module-qtquick-window2 (>= 5.3),
17- qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),
18+ qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.3),
19 qtdeclarative5-ubuntu-web-plugin (= ${binary:Version}),
20 qtdeclarative5-unity-action-plugin,
21 Replaces: webbrowser-app-assets,
22@@ -65,7 +65,7 @@
23 qml-module-qtquick2 (>= 5.4),
24 qml-module-qtquick-window2 (>= 5.3),
25 qtdeclarative5-accounts-plugin,
26- qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),
27+ qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.3),
28 qtdeclarative5-ubuntu-web-plugin (= ${binary:Version}),
29 qtdeclarative5-unity-action-plugin,
30 unity-webapps-qml,
31@@ -83,7 +83,7 @@
32 ${shlibs:Depends},
33 qml-module-qtquick2 (>= 5.4),
34 qml-module-qtquick-window2 (>= 5.3),
35- qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),
36+ qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.3),
37 qtdeclarative5-ubuntu-web-plugin (= ${binary:Version}),
38 Replaces: webbrowser-app (<< 0.21)
39 Breaks: webbrowser-app (<< 0.21)
40@@ -100,7 +100,7 @@
41 liboxideqt-qmlplugin (>= 1.6),
42 qml-module-qtquick2 (>= 5.4),
43 qml-module-qtquick-window2 (>= 5.3),
44- qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),
45+ qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.3),
46 qtdeclarative5-ubuntu-web-plugin-assets (>= ${source:Version}),
47 Description: Ubuntu web QML plugin
48 A standalone QML plugin that contains the WebView component,

Subscribers

People subscribed via source and target branches

to status/vote changes: