Merge lp:~seb128/indicator-appmenu/dont-recommends-qt4 into lp:indicator-appmenu/15.10

Proposed by Sebastien Bacher on 2016-02-29
Status: Approved
Approved by: Sebastien Bacher on 2016-03-18
Approved revision: 277
Proposed branch: lp:~seb128/indicator-appmenu/dont-recommends-qt4
Merge into: lp:indicator-appmenu/15.10
Diff against target: 14 lines (+1/-1)
1 file modified
debian/control (+1/-1)
To merge this branch: bzr merge lp:~seb128/indicator-appmenu/dont-recommends-qt4
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve on 2016-03-17
Ted Gould (community) 2016-02-29 Approve on 2016-03-17
PS Jenkins bot continuous-integration Needs Fixing on 2016-02-29
Review via email: mp+287501@code.launchpad.net

Commit Message

Have appmenu-qt as a Suggests rather than a Recommends, there are no
qt4 clients installed by default on the current Ubuntu image which means
there is no need to keep the integration and runtime installed

Description of the Change

Have appmenu-qt as a Suggests rather than a Recommends, there are no
qt4 clients installed by default on the current Ubuntu image which means
there is no need to keep the integration and runtime installed

To post a comment you must log in.
Ted Gould (ted) wrote :

This looks like a good change to me.

review: Approve
review: Approve
Sebastien Bacher (seb128) wrote :

changing back to work in progress, there are other qt4 depends still on the iso and no point stop installing this one while the others are still there, if the lib are on the iso we can as well keep the integration complete

Unmerged revisions

277. By Sebastien Bacher on 2016-02-29

Have appmenu-qt as a Suggests rather than a Recommends, there are no
qt4 clients installed by default on the current Ubuntu image which means
there is no need to keep the integration and runtime installed

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-10-28 16:49:28 +0000
3+++ debian/control 2016-02-29 17:33:41 +0000
4@@ -33,9 +33,9 @@
5 Recommends: indicator-applet | indicator-renderer,
6 unity-gtk2-module,
7 unity-gtk3-module,
8- appmenu-qt,
9 appmenu-qt5,
10 jayatana,
11+Suggests: appmenu-qt,
12 Description: Indicator for application menus.
13 .
14 This package provides support for application menus.

Subscribers

People subscribed via source and target branches