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

Proposed by Sebastien Bacher
Status: Approved
Approved by: Sebastien Bacher
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
Ted Gould (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
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.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Ted Gould (ted) wrote :

This looks like a good change to me.

review: Approve
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) :
review: Approve
Revision history for this message
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

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