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


Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to