Marble-qt does not create menu entry

[Evan Derickson]

Installing the "marble-qt" package does not create a .desktop file in /usr/share/applications, and therefore does not create a menu entry that can be accessed in the dash (Ubuntu) or application menu (Xubuntu, Lubuntu).

ProblemType: Bug
DistroRelease: Ubuntu 13.10
Package: marble-qt 4:4.11.2-0ubuntu1
ProcVersionSignature: Ubuntu 3.11.0-12.19-generic 3.11.3
Uname: Linux 3.11.0-12-generic x86_64
ApportVersion: 2.12.5-0ubuntu2.1
Architecture: amd64
Date: Fri Oct 25 14:07:16 2013
EcryptfsInUse: Yes
InstallationDate: Installed on 2013-10-25 (0 days ago)
InstallationMedia: Ubuntu 13.10 "Saucy Salamander" - Release amd64 (20131016.1)
MarkForUpload: True
ProcEnviron:
 LANGUAGE=en_US
 PATH=(custom, no user)
 XDG_RUNTIME_DIR=<set>
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: marble
UpgradeStatus: No upgrade log present (probably fresh install)

[Bbeschow (bbeschow)]

No .desktop files are installed for marble-qt (on desktop) and marble-mobile (on Maemo5). So in a Qt-only build there is no desktop file installed at all to start marble.

Reproducible: Always

Steps to Reproduce:
1. Build and install Marble in Qt-only mode
Actual Results:
No desktop files are installed to start marble-qt

Expected Results:
marble-qt and possibly marble-mobile should have their own .desktop files

[Bbeschow (bbeschow)]

Git commit 0a88e37a10f083335a8e28df584e27d85a99eec3 by Bernhard Beschow.
Committed on 08/11/2013 at 21:49.
Pushed by beschow into branch 'master'.

install a .desktop file for each Marble variant (Qt, Mobile, KDE)

It's technically possible to install all .desktop files in parallel.
However, since all files have the same user-visible content, multiple seemingly identical entries will appear in the XDG menu which will start different variants of Marble.
Hence, we need to rely on packagers to select the appropriate .desktop files for the packaged Marble binaries.
REVIEW: 114054

M +2 -0 src/apps/marble-mobile/CMakeLists.txt
A +94 -0 src/apps/marble-mobile/marble-mobile.desktop
M +2 -0 src/apps/marble-qt/CMakeLists.txt
A +95 -0 src/apps/marble-qt/marble-qt.desktop

http://commits.kde.org/marble/0a88e37a10f083335a8e28df584e27d85a99eec3

[Bbeschow (bbeschow)]

Git commit ce1ab4a9203578e634be1e2926e8c7ee03109ad9 by Bernhard Beschow.
Committed on 08/11/2013 at 21:49.
Pushed by beschow into branch 'KDE/4.12'.

install a .desktop file for each Marble variant (Qt, Mobile, KDE)

It's technically possible to install all .desktop files in parallel.
However, since all files have the same user-visible content, multiple seemingly identical entries will appear in the XDG menu which will start different variants of Marble.
Hence, we need to rely on packagers to select the appropriate .desktop files for the packaged Marble binaries.
(cherry picked from commit 0a88e37a10f083335a8e28df584e27d85a99eec3)
REVIEW: 114054
FIXED-IN: 1.7.0 (KDE 4.12.0)

M +2 -0 src/apps/marble-mobile/CMakeLists.txt
A +94 -0 src/apps/marble-mobile/marble-mobile.desktop
M +2 -0 src/apps/marble-qt/CMakeLists.txt
A +95 -0 src/apps/marble-qt/marble-qt.desktop

http://commits.kde.org/marble/ce1ab4a9203578e634be1e2926e8c7ee03109ad9

[Harald Sitter (apachelogger)]

 Hi there!

Thanks for reporting this bug! Your bug seems to be a problem with the KDE program itself, and not with our KDE packages. While we appreciate your issue, it would be better if it was tracked at https://bugs.kde.org, so that the KDE developers can deal with this speedily and have direct communication with you as the reporter for more effective debugging.

Thanks!

[Evan Derickson (derickson-e)]

Harald, I do believe that this issue is with the packaging and *not* with the upstream program. The "marble-qt" package, which is a variant of the "marble" package built without KDE dependencies, does not create a .desktop file in /usr/share/applications.

As the "marble" package does not share this issue, and both packages are built from the same source, this is a packaging issue and would not be appropriate for the upstream bug tracker.

[Harald Sitter (apachelogger)]

The qtonly binary has a different name so one desktop file would not work for both binaries, which makes it an upstream issue as they simply have no desktop file that would work with the qtonly marble build.

[Thiago Macieira (thiago-kde)]

This commit broke the build for Qt-only Mac build.

CMake Error at src/apps/marble-qt/CMakeLists.txt:80 (install):
  install PROGRAMS given no DESTINATION!

CMake Error at src/apps/marble-mobile/CMakeLists.txt:27 (install):
  install PROGRAMS given no DESTINATION!

CMake Error at src/apps/marble-touch/CMakeLists.txt:40 (install):
  install PROGRAMS given no DESTINATION!

The variable APPS_INSTALL_DIR is not initialised for that case.

[Dennis Nienhüser (earthwings)]

Thanks for reporting.

Mohammed, can you chime in? :-)

[Bbeschow (bbeschow)]

Git commit 63bce6898862535aa7efa5109835cbb684548380 by Bernhard Beschow.
Committed on 30/11/2013 at 13:29.
Pushed by beschow into branch 'master'.

don't attempt to install .desktop files on platforms that don't support them

M +3 -2 src/apps/marble-mobile/CMakeLists.txt
M +3 -2 src/apps/marble-qt/CMakeLists.txt
M +3 -1 src/apps/marble-touch/CMakeLists.txt

http://commits.kde.org/marble/63bce6898862535aa7efa5109835cbb684548380

[Bbeschow (bbeschow)]

Git commit 7ab9e931af2810ebd55c3aacd4de376ff57c45a3 by Bernhard Beschow.
Committed on 30/11/2013 at 13:29.
Pushed by beschow into branch 'KDE/4.12'.

don't attempt to install .desktop files on platforms that don't support them
(cherry picked from commit 63bce6898862535aa7efa5109835cbb684548380)

M +3 -2 src/apps/marble-mobile/CMakeLists.txt
M +3 -2 src/apps/marble-qt/CMakeLists.txt
M +3 -1 src/apps/marble-touch/CMakeLists.txt

http://commits.kde.org/marble/7ab9e931af2810ebd55c3aacd4de376ff57c45a3

[Scarpino (scarpino)]

Why you don't read MOBILE and QTONLY variables before you install the desktop file for that specific version?
e.g. if (MOBILE) install marble-mobile.desktop

[Peter (auxsvr-gmail)]

I'm not sure whether I should open a new report for this: on KDE 4.12.2, opensuse 12.3, there are 4 items in the kickoff menu named "marble" with identical information and icons, making selection among marble-kde, marble-qt, marble-touch and marble-mobile a matter of luck. Wouldn't it be preferable if, for example, the GenericName included a word to distinguish among the variants?

[Evan Derickson (derickson-e)]

Fixed in Ubuntu 14.04

[Nienhueser (nienhueser)]

Git commit 407a29c460682beb68ddb3961138870a67c684d0 by Dennis Nienhüser.
Committed on 10/09/2015 at 15:27.
Pushed by nienhueser into branch 'master'.

Remove .desktop files and do not install now outdated applications

This affects the Maemo (marble-mobile) and MeeGo (marble-touch) apps.
Android (marble-maps) will replace them. Still compiling marble-mobile
and marble-touch though as they are useful for further development.
Related: bug 333838

M +0 -4 src/apps/marble-mobile/CMakeLists.txt
D +0 -99 src/apps/marble-mobile/marble-mobile.desktop
M +0 -5 src/apps/marble-touch/CMakeLists.txt
D +0 -99 src/apps/marble-touch/marble-touch.desktop

http://commits.kde.org/marble/407a29c460682beb68ddb3961138870a67c684d0

[Nienhueser (nienhueser)]

Git commit 489c0ec8a6a51813c4f9e7c87e2b088c34b32a63 by Dennis Nienhüser.
Committed on 10/09/2015 at 15:27.
Pushed by nienhueser into branch 'master'.

Add a .desktop file for marble-maps, limit installation

The app and .desktop file are only installed on ANDROID by default.
The cmake option MARBLE_INSTALL_ANDROID_APPS can be used to install
marble-maps on other targets as well.
Related: bug 333838

M +6 -1 src/apps/marble-maps/CMakeLists.txt
A +9 -0 src/apps/marble-maps/marble-maps.desktop

http://commits.kde.org/marble/489c0ec8a6a51813c4f9e7c87e2b088c34b32a63

[Nienhueser (nienhueser)]

Git commit e21bb08a6b9b04aa9c6bc17e9b74e2e8827e5aeb by Dennis Nienhüser.
Committed on 10/09/2015 at 15:27.
Pushed by nienhueser into branch 'master'.

Add Comment entries, distinguish Qt/KDE versions via their name
Related: bug 351972, bug 333838

M +2 -1 src/apps/marble-kde/marble.desktop
M +1 -0 src/apps/marble-maps/marble-maps.desktop
M +1 -0 src/apps/marble-qt/marble-qt.desktop

http://commits.kde.org/marble/e21bb08a6b9b04aa9c6bc17e9b74e2e8827e5aeb