Merge lp:~mitya57/indicator-appmenu/drop-appmenu-qt5 into lp:indicator-appmenu

Proposed by Dmitry Shachnev on 2016-09-27
Status: Merged
Approved by: Timo Jyrinki on 2016-09-27
Approved revision: 277
Merged at revision: 277
Proposed branch: lp:~mitya57/indicator-appmenu/drop-appmenu-qt5
Merge into: lp:indicator-appmenu
Diff against target: 11 lines (+0/-1)
1 file modified
debian/control (+0/-1)
To merge this branch: bzr merge lp:~mitya57/indicator-appmenu/drop-appmenu-qt5
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) 2016-09-27 Approve on 2016-09-28
Timo Jyrinki 2016-09-27 Approve on 2016-09-27
Indicator Applet Developers 2016-09-27 Pending
Review via email: mp+306862@code.launchpad.net

Commit message

Drop appmenu-qt5 recommendation.

Description of the change

We want to not ship appmenu-qt5 in yakkety by default.

To post a comment you must log in.
Timo Jyrinki (timo-jyrinki) wrote :

Yes, not needed anymore and preferable not to have it installed.

review: Approve
Marco Trevisan (Treviño) (3v1n0) wrote :

Ok!

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-10-28 16:49:28 +0000
+++ debian/control 2016-09-27 12:19:31 +0000
@@ -34,7 +34,6 @@
34 unity-gtk2-module,34 unity-gtk2-module,
35 unity-gtk3-module,35 unity-gtk3-module,
36 appmenu-qt,36 appmenu-qt,
37 appmenu-qt5,
38 jayatana,37 jayatana,
39Description: Indicator for application menus.38Description: Indicator for application menus.
40 .39 .

Subscribers

People subscribed via source and target branches