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
=== modified file 'debian/control'
--- debian/control 2015-08-11 19:05:10 +0000
+++ debian/control 2015-08-16 23:16:32 +0000
@@ -21,7 +21,7 @@
21 qtbase5-private-dev,21 qtbase5-private-dev,
22 qtdeclarative5-dev,22 qtdeclarative5-dev,
23 qtdeclarative5-private-dev,23 qtdeclarative5-private-dev,
24 qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),24 qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.3),
25 xvfb,25 xvfb,
26Standards-Version: 3.9.526Standards-Version: 3.9.5
27Homepage: https://launchpad.net/webbrowser-app27Homepage: https://launchpad.net/webbrowser-app
@@ -42,7 +42,7 @@
42 qml-module-qtquick2 (>= 5.4),42 qml-module-qtquick2 (>= 5.4),
43 qml-module-qtquick-dialogs,43 qml-module-qtquick-dialogs,
44 qml-module-qtquick-window2 (>= 5.3),44 qml-module-qtquick-window2 (>= 5.3),
45 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 (= ${binary:Version}),46 qtdeclarative5-ubuntu-web-plugin (= ${binary:Version}),
47 qtdeclarative5-unity-action-plugin,47 qtdeclarative5-unity-action-plugin,
48Replaces: webbrowser-app-assets,48Replaces: webbrowser-app-assets,
@@ -65,7 +65,7 @@
65 qml-module-qtquick2 (>= 5.4),65 qml-module-qtquick2 (>= 5.4),
66 qml-module-qtquick-window2 (>= 5.3),66 qml-module-qtquick-window2 (>= 5.3),
67 qtdeclarative5-accounts-plugin,67 qtdeclarative5-accounts-plugin,
68 qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),68 qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.3),
69 qtdeclarative5-ubuntu-web-plugin (= ${binary:Version}),69 qtdeclarative5-ubuntu-web-plugin (= ${binary:Version}),
70 qtdeclarative5-unity-action-plugin,70 qtdeclarative5-unity-action-plugin,
71 unity-webapps-qml,71 unity-webapps-qml,
@@ -83,7 +83,7 @@
83 ${shlibs:Depends},83 ${shlibs:Depends},
84 qml-module-qtquick2 (>= 5.4),84 qml-module-qtquick2 (>= 5.4),
85 qml-module-qtquick-window2 (>= 5.3),85 qml-module-qtquick-window2 (>= 5.3),
86 qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),86 qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.3),
87 qtdeclarative5-ubuntu-web-plugin (= ${binary:Version}),87 qtdeclarative5-ubuntu-web-plugin (= ${binary:Version}),
88Replaces: webbrowser-app (<< 0.21)88Replaces: webbrowser-app (<< 0.21)
89Breaks: webbrowser-app (<< 0.21)89Breaks: webbrowser-app (<< 0.21)
@@ -100,7 +100,7 @@
100 liboxideqt-qmlplugin (>= 1.6),100 liboxideqt-qmlplugin (>= 1.6),
101 qml-module-qtquick2 (>= 5.4),101 qml-module-qtquick2 (>= 5.4),
102 qml-module-qtquick-window2 (>= 5.3),102 qml-module-qtquick-window2 (>= 5.3),
103 qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3),103 qtdeclarative5-ubuntu-ui-toolkit-plugin (>= 1.3) | qtdeclarative5-ubuntu-ui-toolkit-plugin-gles (>= 1.3),
104 qtdeclarative5-ubuntu-web-plugin-assets (>= ${source:Version}),104 qtdeclarative5-ubuntu-web-plugin-assets (>= ${source:Version}),
105Description: Ubuntu web QML plugin105Description: Ubuntu web QML plugin
106 A standalone QML plugin that contains the WebView component,106 A standalone QML plugin that contains the WebView component,

Subscribers

People subscribed via source and target branches

to status/vote changes: