make package "*** Error code 1"

Fri Jun 8 10:32:49 UTC 2018

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

On Fri, 8 Jun 2018 05:58:01 -0400
Matthew Seaman <matthew at FreeBSD.org> wrote:

> On 07/06/2018 21:17, Eugene Grosbein wrote:
> > 08.06.2018 8:12, duckmanjbr wrote:
> >   
> >> do-install:
> >> 	${MKDIR} ${STAGEDIR}${PREFIX}/pkg
> >> 	${MKDIR} ${STAGEDIR}${PREFIX}/bin/localrepo
> >> 	${MKDIR} ${STAGEDIR}/etc/inc/priv
> >> 	${MKDIR} ${STAGEDIR}${DATADIR}
> >> 	${INSTALL_DATA} -m 0644
> >> ${FILESDIR}${PREFIX}/pkg/localrepo.xml \ ${STAGEDIR}${PREFIX}/pkg
> >> 	${INSTALL_DATA} ${FILESDIR}${PREFIX}/pkg/localrepo.inc \
> >> 		${STAGEDIR}${PREFIX}/pkg
> >> 	${INSTALL_DATA}
> >> ${FILESDIR}${PREFIX}/bin/localrepo/localrepo.conf \
> >> ${STAGEDIR}${PREFIX}/bin/localrepo ${INSTALL_DATA}
> >> ${FILESDIR}/etc/inc/priv/localrepo.priv.inc \
> >> ${STAGEDIR}/etc/inc/priv ${INSTALL_DATA}
> >> ${FILESDIR}${DATADIR}/info.xml \ ${STAGEDIR}${DATADIR}/info.xml
> >> 	@${REINPLACE_CMD} -i '' -e
> >> "s|%%PKGVERSION%%|${PKGVERSION}|" \
> >>
> >> .include <bsd.port.mk>  
> > 
> > Here is obvious bug: the line with REINPLACE_CMD has continuation
> > backslash without any continuation text and without file name to
> > process. Fix it.  
> 
> Actually, since the next line is a blank line, that's going to be
> pretty harmless.  It should be fixed as a matter of style though.
> 
> The thing that is actually biting the OP is that ${REINPLACE_CMD}
> expands to this:
> 
> % make -V REINPLACE_CMD
> /usr/bin/sed -i.bak
> 
> So you're already passing a '-i' flag to sed(1), and it's the
> additional -i '' on your command line that sed(1) doesn't like.

That may be wrong, but the complaint is about using -i on standard
input. There is definitely a missing filename.