Re: make package "*** Error code 1"

2018-06-08 Thread Eugene Grosbein
08.06.2018 20:12, duckmanjbr wrote:

> 1st off,  thanks everyone for helping me out!  There was indeed a missing 
> filename and I believe I’ve remedied that.  I’m moving along but now getting 
> a "missing property prefix” error.  Am I missing a log file or something to 
> help trouble shoot these errors?  I’d like to piece it together to a working 
> state before I offer it as a pull request to pfSense/FreeBSD-ports. 
> 
> build_error:
> root@:~/pfSense-pkg-localrepo # make package
> ===>  License APACHE20 accepted by the user
> ===>   pfSense-pkg-LocalRepo-0.1_1 depends on file: /usr/local/sbin/pkg - 
> found
> ===> Fetching all distfiles required by pfSense-pkg-LocalRepo-0.1_1 for 
> building
> ===>  Extracting for pfSense-pkg-LocalRepo-0.1_1
> /bin/mkdir -p /root/pfSense-pkg-localrepo/work/pfSense-pkg-LocalRepo-0.1
> ===>  Patching for pfSense-pkg-LocalRepo-0.1_1
> ===>  Configuring for pfSense-pkg-LocalRepo-0.1_1
> ===>  Staging for pfSense-pkg-LocalRepo-0.1_1
> ===>   Generating temporary packing list
> /bin/mkdir -p /root/pfSense-pkg-localrepo/work/stage/usr/local/pkg
> /bin/mkdir -p /root/pfSense-pkg-localrepo/work/stage/usr/local/bin/localrepo
> /bin/mkdir -p /root/pfSense-pkg-localrepo/work/stage/etc/inc/priv
> /bin/mkdir -p 
> /root/pfSense-pkg-localrepo/work/stage/share/pfSense-pkg-LocalRepo
> install  -m 0644 -m 0644 
> /root/pfSense-pkg-localrepo/files/usr/local/pkg/localrepo.xml  
> /root/pfSense-pkg-localrepo/work/stage/usr/local/pkg
> install  -m 0644 
> /root/pfSense-pkg-localrepo/files/usr/local/pkg/localrepo.inc  
> /root/pfSense-pkg-localrepo/work/stage/usr/local/pkg
> install  -m 0644 
> /root/pfSense-pkg-localrepo/files/usr/local/bin/localrepo/localrepo.conf  
> /root/pfSense-pkg-localrepo/work/stage/usr/local/bin/localrepo
> install  -m 0644 
> /root/pfSense-pkg-localrepo/files/etc/inc/priv/localrepo.priv.inc  
> /root/pfSense-pkg-localrepo/work/stage/etc/inc/priv
> install  -m 0644 
> /root/pfSense-pkg-localrepo/files/usr/local/share/pfSense-pkg-LocalRepo/info.xml
>   /root/pfSense-pkg-localrepo/work/stage/share/pfSense-pkg-LocalRepo
> > Compressing man pages (compress-man)
> ===>  Building package for pfSense-pkg-LocalRepo-0.1_1
> [: =: unexpected operator
> pkg-static: Invalid package: object has missing property prefix
> pkg-static: the package is not valid
> *** Error code 1

I guess, your files/pkg-install.in file may have bugs.
And this seems to be very pfSence-specific, you probably get more attention 
asking pfSense people.

___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"


Re: make package "*** Error code 1"

2018-06-08 Thread duckmanjbr
1st off,  thanks everyone for helping me out!  There was indeed a missing 
filename and I believe I’ve remedied that.  I’m moving along but now getting a 
"missing property prefix” error.  Am I missing a log file or something to help 
trouble shoot these errors?  I’d like to piece it together to a working state 
before I offer it as a pull request to pfSense/FreeBSD-ports. 

build_error:
root@:~/pfSense-pkg-localrepo # make package
===>  License APACHE20 accepted by the user
===>   pfSense-pkg-LocalRepo-0.1_1 depends on file: /usr/local/sbin/pkg - found
===> Fetching all distfiles required by pfSense-pkg-LocalRepo-0.1_1 for building
===>  Extracting for pfSense-pkg-LocalRepo-0.1_1
/bin/mkdir -p /root/pfSense-pkg-localrepo/work/pfSense-pkg-LocalRepo-0.1
===>  Patching for pfSense-pkg-LocalRepo-0.1_1
===>  Configuring for pfSense-pkg-LocalRepo-0.1_1
===>  Staging for pfSense-pkg-LocalRepo-0.1_1
===>   Generating temporary packing list
/bin/mkdir -p /root/pfSense-pkg-localrepo/work/stage/usr/local/pkg
/bin/mkdir -p /root/pfSense-pkg-localrepo/work/stage/usr/local/bin/localrepo
/bin/mkdir -p /root/pfSense-pkg-localrepo/work/stage/etc/inc/priv
/bin/mkdir -p /root/pfSense-pkg-localrepo/work/stage/share/pfSense-pkg-LocalRepo
install  -m 0644 -m 0644 
/root/pfSense-pkg-localrepo/files/usr/local/pkg/localrepo.xml  
/root/pfSense-pkg-localrepo/work/stage/usr/local/pkg
install  -m 0644 /root/pfSense-pkg-localrepo/files/usr/local/pkg/localrepo.inc  
/root/pfSense-pkg-localrepo/work/stage/usr/local/pkg
install  -m 0644 
/root/pfSense-pkg-localrepo/files/usr/local/bin/localrepo/localrepo.conf  
/root/pfSense-pkg-localrepo/work/stage/usr/local/bin/localrepo
install  -m 0644 
/root/pfSense-pkg-localrepo/files/etc/inc/priv/localrepo.priv.inc  
/root/pfSense-pkg-localrepo/work/stage/etc/inc/priv
install  -m 0644 
/root/pfSense-pkg-localrepo/files/usr/local/share/pfSense-pkg-LocalRepo/info.xml
  /root/pfSense-pkg-localrepo/work/stage/share/pfSense-pkg-LocalRepo
> Compressing man pages (compress-man)
===>  Building package for pfSense-pkg-LocalRepo-0.1_1
[: =: unexpected operator
pkg-static: Invalid package: object has missing property prefix
pkg-static: the package is not valid
*** Error code 1

Stop.
make: stopped in /root/pfSense-pkg-localrepo
root@:~/pfSense-pkg-localrepo #

makefile:
# $FreeBSD$

PORTNAME=   pfSense-pkg-LocalRepo
PORTVERSION= 0.1
PORTREVISION= 1
CATEGORIES= sysutils
MASTER_SITES=   # empty
DISTFILES=  # empty
EXTRACT_ONLY=   # empty

MAINTAINER=
COMMENT=Create local repositories on pfSense

LICENSE=APACHE20

NO_BUILD=   yes
NO_MTREE=   yes

SUB_FILES=  pkg-install pkg-deinstall
SUB_LIST=   PORTNAME=${PORTNAME}

do-extract:
${MKDIR} ${WRKSRC}

do-install:
# {STAGEDIR}= /root/pfSense-pkg-localrepo/work/stage
# {PREFIX}= /usr/local
# {DATADIR}= /usr/local/share/pfSense-pkg-LocalRepo
${MKDIR} ${STAGEDIR}${PREFIX}/pkg
${MKDIR} ${STAGEDIR}${PREFIX}/bin/localrepo
${MKDIR} ${STAGEDIR}/etc/inc/priv
${MKDIR} ${STAGEDIR}/share/pfSense-pkg-LocalRepo
${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}/share/pfSense-pkg-LocalRepo
@${REINPLACE_CMD} -i '' -e "s|%%PKGVERSION%%|${PKGVERSION}|" \
${STAGEDIR}/share/pfSense-pkg-LocalRepo/info.xml

.include 

pkg-plist:
usr/local/pkg/localrepo.xml
usr/local/pkg/localrepo.inc
usr/local/bin/localrepo/localrepo.conf
/etc/inc/priv/localrepo.priv.inc
%%DATADIR%%/info.xml
@dir /etc/inc/priv



> On Jun 8, 2018, at 7:13 AM, Eugene Grosbein  wrote:
> 
> 08.06.2018 16:58, Matthew Seaman wrote:
> 
@${REINPLACE_CMD} -i '' -e "s|%%PKGVERSION%%|${PKGVERSION}|" \
 
 .include 
>>> 
>>> 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.
> 
> No. A filename is still missing.
> 
> ___
> freebsd-ports@freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/freebsd-ports
> To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"

___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"


Re: make package "*** Error code 1"

2018-06-08 Thread Eugene Grosbein
08.06.2018 16:58, Matthew Seaman wrote:

>>> @${REINPLACE_CMD} -i '' -e "s|%%PKGVERSION%%|${PKGVERSION}|" \
>>>
>>> .include 
>>
>> 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.

No. A filename is still missing.

___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"


Re: make package "*** Error code 1"

2018-06-08 Thread Bob Eager
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256

On Fri, 8 Jun 2018 05:58:01 -0400
Matthew Seaman  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   
> > 
> > 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.

-BEGIN PGP SIGNATURE-

iQEzBAEBCAAdFiEEVgdI2KeVldPAhUYaKBdf2az8e6gFAlsaWUwACgkQKBdf2az8
e6hyfggAi80w3j2VL0Bnv6Hvh0cs6oxS+HlCF4WV+VU1sAxDNPz6Sd799FuCQBpJ
n2/E4F+Cf1DwdPgnE/ks+VDi3YDH6th6pUqSvNfcJK03qsqt9c0Td8k5W+KAMGVu
jPg+nte+2eCQLtTeW2tSE6oxw1TYlhfo0avyQ2ZtJJoBvo1myk52TmrRFE+qzdgm
17NvyU0gmBtMNS89AL0AiHHcHXvRCc5szLtNF9PoCyVCxu26Q1l24IR8SiDhz2UU
gJXKRsBUy91T2uCBvlZMrH9GIcCDHwVJxhVBRXL3/PFWOxWgMOSvOIt/wjrk4Knh
jTCCaoK0Babb1CQo5DuRG6m8IN9p9g==
=H8JY
-END PGP SIGNATURE-
___
freebsd-ports@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscr...@freebsd.org"


Re: make package "*** Error code 1"

2018-06-08 Thread 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 
> 
> 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




signature.asc
Description: OpenPGP digital signature