Merge lp:~ubuntu-mate-dev/landscape-client/show-in-mate into lp:~landscape/landscape-client/trunk

[Martin Wimpress ]

Let the landscape-client-ui show up in Ubuntu MATE

I'd like to include the landscape-client in Ubuntu MATE but the icon is not visible under MATE desktop. This merge proposal simply makes the landscape-client desktop entry visible under MATE.

[Chris Glass (tribaal)]

(Adding a second slot for Landscape developers as per landscape policy)

This MP is in the right place, we'll look at it as soon as possible. Thanks a lot!

[Chris Glass (tribaal)]

Thanks for the MP, this looks good (at least, harmless), however:

I installed Ubuntu Mate 15.04 in a KVM, and applied this diff to /usr/share/app-install/desktop/landscape-client-ui-install:landscape-client-settings.desktop in order to assert that it works.

Being a complete MATE newbie - where should I look for the landscape-client entry? Most of my (arguably short) discovery testing by poking around in the menus didn't show anything new in the system settings panel. Is there something I need to regenerate for this to show?

How can I check this actually works?

Thanks for you help.

- Chris

review: Needs Information

[Martin Wimpress  (flexiondotorg)]

IIRC it should appear under the top panel menu System -> Preferences -> Other.

[Chris Glass (tribaal)]

> IIRC it should appear under the top panel menu System -> Preferences -> Other.

I looked there and in every other menu entry, to no avail.

[Martin Wimpress  (flexiondotorg)]

I installed "landscape-client-ui-install" then editted /usr/share/applications/landscape-client-settings.desktop and added MATE; to OnlyShowIn. After running "sudo update-desktop-database" the Lanscape Service menu entry appeared in System - Preferences - Other. Hope that helps :-)

[Chris Glass (tribaal)]

Verified this works in a new 15.04 MATE install with the provided instructions, no idea what went wrong the first time.

Thanks for you work!

review: Approve

[Geoff Teale (tealeg)]

+1 Also working here.

review: Approve

[Chris Glass (tribaal)]

This patch was merged in landscape-client trunk.

Thanks a lot for your contribution!

We will roll it out with the rest of the changes next time we release a landscape-client package (we usually SRU the update back ourselves).

=== modified file 'applications/landscape-client-settings.desktop.in'
--- applications/landscape-client-settings.desktop.in	2014-01-15 19:25:15 +0000
+++ applications/landscape-client-settings.desktop.in	2015-07-20 10:02:33 +0000
@@ -7,7 +7,7 @@
 Type=Application
 StartupNotify=true
 Categories=GNOME;GTK;Settings;X-GNOME-SystemSettings;X-GNOME-Settings-Panel;X-Unity-Settings-Panel;
-OnlyShowIn=GNOME;Unity;
+OnlyShowIn=GNOME;Unity;MATE;
 X-Unity-Settings-Panel=landscape
 X-GNOME-Settings-Panel=landscape
 X-Ubuntu-Gettext-Domain=landscape-client