getopt() returns int' according to the standard, but was assigned to char'

variable, and this can break the == -1 check.  It works on i386, but not on
PowerPC, where `char' is unsigned by default.

Use variable of correct type (int) to hold the return value of getopt(3).
This commit is contained in:
Alexey Dokuchaev 2015-09-08 12:07:09 +00:00
parent 5ca09a9d3b
commit 8da335f97b
Notes: svn2git 2021-03-31 03:12:20 +00:00
svn path=/head/; revision=396375

View file

@ -30,10 +30,6 @@ OPTIONS_DEFAULT= SKEM_NETINET
.include <bsd.port.options.mk>
.if ${ARCH} == "powerpc"
BROKEN= Does not compile on powerpc
.endif
MAKE_ENV+= PTHREAD_LIBS="-lpthread"
.for o in ${PORT_OPTIONS:MSKEM_*}
@ -42,4 +38,7 @@ MAKE_ARGS+= -D$o
. endif
.endfor
post-patch:
@${REINPLACE_CMD} -E 's/char[[:blank:]]+c;/int c;/' ${WRKSRC}/milter.c
.include <bsd.port.mk>