On Mon, Jul 16, 2007 at 11:43:17PM +0200, Antoine Jacoutot wrote:
> On Mon, 16 Jul 2007, Landry Breuil wrote:
> >>- --prefix=${LOCALBASE} looks wrong
> >
> >Changed to /usr/local
>
> Huh? No, it should be --prefix=${PREFIX}
Okay, now i understand (thanks kili for the explanations)
> >ps: i still think my patching of man/Makefile is ugly, please someone look
> >at it ! :)
>
> Well you could rewrite the entire target in the port Makefile. Their
> hand-crafted Makefiles are weird anyway.
That's what i did, dunno if it was the best way to do it.
Patch Makefile to remove man/ from SUBDIRS, and added the following hook :
post-install:
@sed -e 's,PREFIX,${LOCALBASE},g' ${WRKSRC}/man/fbpanel.1 > \
${WRKBUILD}/man/fbpanel.1.fixed
${INSTALL_MAN} ${WRKBUILD}/man/fbpanel.1.fixed \
${PREFIX}/man/man1/fbpanel.1
Final new tarball available here :
http://gcu.info/~gaston/ports/fbpanel-4.11_3.tar.gz
Comments, Ok ?
Thanks,
Landry
