Install jamulus.desktop #331
Conversation
Because this assumes the binary is called jamulus, and qmake doesn't have a mechanism for string expansion, only do so if noupcasename is set.
| APPSDIR = $$absolute_path($$APPSDIR, $$PREFIX) | ||
| desktop.path = $$APPSDIR | ||
| desktop.files = distributions/jamulus.desktop | ||
| # the .desktop file assumes the binary is called jamulus |
There was a problem hiding this comment.
It should be possible to rename the desktop file end with .in, put $$TARGET in the Exec file and then use QMAKE_SUBSTITUTES += distributions/jamulus.desktop.in.
See https://martin.rpdev.net/2017/05/10/creating-config-header-files-using-qmake.html or https://github.com/detrout/debian-signon-ui/blob/c03c68965f92285d3d5341b45414473081ed5c1b/src/signon-ui.pro#L105-L109.
There was a problem hiding this comment.
Ahh! That is... extremely well hidden in the qmake documentation.
| # the .desktop file assumes the binary is called jamulus | ||
| contains(CONFIG, "noupcasename") { | ||
| INSTALLS += desktop | ||
| } | ||
| } |
There was a problem hiding this comment.
We probably also need to install the icon, see #247 (comment).
seems to work! |
As suggested in #223.
Because jamulus.desktop assumes the binary is called jamulus, and qmake doesn't have a mechanism for string expansion, only install the file so if noupcasename is set.