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

[Matthew Brush]

@codebrainz 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

Can't you just do this:

```diff
- Exec=geany %F
+ Exec=@bindir@/geany %F
```

In [the `geany.desktop.in` file](https://github.com/geany/geany/blob/d9f8cdbad58d09f0c18ca8acccb49209263018f0/geany.desktop.in#L7) or similar?

I'm pretty sure if you added it to `AC_CONFIG_FILES` it would get the substitutions. But maybe I'm missing something of Geany's Autotools.

-- 
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#pullrequestreview-570165514
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20210117/e35e35d2/attachment.htm>