On Wed, Jan 19, 2011 at 06:04:17AM +0300, Vadim Zhukov wrote:
> On 18 January 2011 ?. 22:09:53 Marco Peereboom wrote:
> > anyone?
>
> Oops, there is packaging problem with .desktop file: pkg-subst replaces
> TRUEPREFIX only in the beginning of line, so the file is actually
> copied with ${TRUEPREFIX}/bin/xxxterm (and so) line itself. Updated
> patch below.
> - @${SUBST_CMD} -c ${WRKSRC}/xxxterm.desktop
> ${PREFIX}/share/applications/xxxterm.desktop
> - ${INSTALL_DATA} ${WRKSRC}/xxxterm.desktop ${PREFIX}/share/applications/
> - ${CHOWN} ${SHAREOWN}:${SHAREGRP}
> $(PREFIX)/share/applications/xxxterm.desktop
> + ${INSTALL_DATA} ${WRKSRC}/xxxterm.desktop
> ${PREFIX}/share/applications/xxxterm.desktop
> + ${SUBST_CMD} ${PREFIX}/share/applications/xxxterm.desktop
It's more wrong. subst was done on the source file and was installed to
prefix, then overwritten by the install_data just after.
you can also just use
${SUBST_CMD} -o ${SHAREOWN} -g ${SHAREGRP} -c ${WRKSRC}/xxxterm.desktop
${PREFIX}/share/applications/xxxterm.desktop
instead of doing install+subst
Landry
