Merge lp:~sil2100/indicator-appmenu/indicator-appmenu-qt5 into lp:indicator-appmenu/13.10

[Łukasz Zemczak]

We're adding Qt5 support to appmenu, let's pull that in case someone would install Qt5. It's safe, since the added packages don't depend on Qt5 directly - they Suggest it instead.

NOTE! There are a few merges that need to get in before this one can be accepted, be sure to check them before approving!

-> https://code.launchpad.net/~sil2100/appmenu-qt/appmenu-qt-multi/+merge/156002
-> https://code.launchpad.net/~sil2100/libdbusmenu-qt/libdbusmenu-qt-multi/+merge/156010
-> https://code.launchpad.net/~sil2100/libdbusmenu-qt/libdbusmenu-qt-inline-and-qt5/+merge/156015

- Problem:

We lack Qt5 appmenu support. We're adding that. We want to Recommend appmenu-qt5 to have it installed in case Qt5 would be installed by anyone.

- Fix:

We add it to Recommends:. This is safe, since appmenu-qt5 and libdbusmenu-qt5 are hacked not to Depend on Qt5. Instead they 'Suggest' on Qt5. This way, we can safely Recommend those packages and feel safe that Qt5 won't get installed when not needed.

- Tests:

N/A

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:236
http://jenkins.qa.ubuntu.com/job/indicator-appmenu-ci/1/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-appmenu-raring-amd64-ci/1

Click here to trigger a rebuild:
http://s-jenkins:8080/job/indicator-appmenu-ci/1/rebuild

review: Approve (continuous-integration)

[Mathieu Trudel-Lapierre (cyphermox)]

Needs fixing: just because this needs to wait for now... otherwise it's good :)

review: Needs Fixing

[Mathieu Trudel-Lapierre (cyphermox)]

Let's land this so we can make it build for daily release.

review: Approve

=== modified file 'debian/changelog'
--- debian/changelog	2013-02-20 09:41:54 +0000
+++ debian/changelog	2013-03-28 16:17:22 +0000
@@ -1,3 +1,10 @@
+indicator-appmenu (13.01.0daily13.02.20-0ubuntu2) UNRELEASED; urgency=low
+
+  * debian/control:
+    - We're recommending the installation of appmenu-qt5 now, which is safe
+
+ -- Łukasz 'sil2100' Zemczak <[e-mail]>  Thu, 28 Mar 2013 15:36:34 +0100
+
 indicator-appmenu (13.01.0daily13.02.20-0ubuntu1) raring; urgency=low
 
   [ Mathieu Trudel-Lapierre ]

=== modified file 'debian/control'
--- debian/control	2013-02-19 17:08:57 +0000
+++ debian/control	2013-03-28 16:17:22 +0000
@@ -36,6 +36,7 @@
             appmenu-gtk,
             appmenu-gtk3,
             appmenu-qt,
+            appmenu-qt5,
 Description: Indicator for application menus.
  .
  This package provides support for application menus.