@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

Reply via email to