On Fri, 12 Dec 2008, Andreas Bihlmaier wrote:
> On Fri, Dec 05, 2008 at 09:36:28AM +0100, Giovanni Bechis wrote:
> > Andreas Bihlmaier ha scritto:
> >> +post-patch:
> >> + @perl -pi -e 's,%%LOCALBASE%%,${LOCALBASE},' ${WRKSRC}/pacpl
> >>
> > I think you should use ${SUBST_CMD} instead of this.
> > Cheers
> > Giovanni
>
> Thanks took me a little help from #openbsd, but after I found out that I
> have to put ${LOCALBASE} into the file, not LOCALBASE. It works.
>
> Updated patch, btw. could somebody commit this if there are no other
> issues :)
> ++KDEDIR = /usr/local
Do not hardcode.
> ++CONF_DIR = $(DESTDIR)$(LOCALBASE)/share/examples/$(PACKAGE_NAME)
> ++PO_DIR = $(DESTDIR)$(LOCALBASE)/share/pacpl/locale
s/LOCALBASE/PREFIX/g here
--
Antoine
