make package "*** Error code 1"

[Matthew Seaman]

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.

	Cheers,

	Matthew


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 963 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freebsd.org/pipermail/freebsd-ports/attachments/20180608/1385251d/attachment.sig>