[Github-comments] [geany/geany] write absolute path to geany bin in desktop file (#2728)

[Colomban Wendling]

@b4n commented on this pull request.



> @@ -37,6 +37,7 @@ uninstall-local:
 
 # manually install some files under another name
 install-data-local:
+	sed -i '/Exec=/c\Exec=@bindir@/geany %F' @top_builddir@/geany.desktop

@codebrainz It's not that simple, because all the `*dir` variables usually expand to placeholders like `${prefix}/bin` and such.  So you need to expand them first, and as mentioned in another comment, it's not super trivial.  Plus, in some situations they can be overridden up until make calls, so it's better to apply them there.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2728#discussion_r559542327
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20210118/f4e8e06b/attachment.htm>