On Thu, Feb 15, 2024 at 09:51:05PM +0300, Kirill Bychkov wrote:
> On Thu, February 8, 2024 14:47, Klemens Nanni wrote:
> > __PREFIX__ expands to PREFIX which includes WRKDIR during fake.
> >
> > One easy fix is to replace it with TRUEPREFIX aka. /usr/local
> > after the fact;  that fixes startup via desktop menu entries.
> >
> > I did not see an easy way to pass the proper prefix to just those
> > .desktop.in files as they seem to be handled during general setup.py
> > install along with all other files.
> >
> > Feedback? OK?
> 
> Hi!

> How about doing that in post-extract stage:

I already committed the diff and did so in install since it makes clear
the the build itself remains unchanged;  doing it in post-extract would
mean that after `make extract' you can't inspect pristine sources.

> diff -u -p -u -p -r1.39 Makefile
> --- Makefile	9 Jan 2024 01:21:47 -0000	1.39
> +++ Makefile	15 Feb 2024 18:50:20 -0000
> @@ -1,6 +1,7 @@
>  COMMENT=		media aggregator and podcast client
> 
>  MODPY_EGG_VERSION=	3.11.4
> +REVISION=		0
> 
>  GH_ACCOUNT=		gpodder
>  GH_PROJECT=		gpodder
> @@ -48,6 +49,9 @@ ALL_TARGET=		messages
> 
>  TEST_TARGET=		unittest
> 
> +post-extract:
> +	sed -i s,__PREFIX__,${TRUEPREFIX}, \
> +		${WRKSRC}/share/applications/*.desktop.in
>  do-install:
>  	cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} DESTDIR=/ ${MAKE_PROGRAM} \
>  		-f ${MAKE_FILE} install