* David Krause <[EMAIL PROTECTED]> [050703 19:29]: > ===> Checking files for py-ldap-2.0.8 > >> python-ldap-2.0.8.tar.gz doesn't seem to exist on this system. > >> Attempting to fetch /usr/ports/distfiles/python-ldap-2.0.8.tar.gz > >> from http://mesh.dl.sourceforge.net/sourceforge/python-ldap/. > >> Size does not match for /usr/ports/distfiles/python-ldap-2.0.8.tar.gz > /bin/sh: test: 30000: unexpected operator/operand > *** Error code 2
It's not that the distfiles have spaces, although it doesn't hurt to check, it still breaks with espie's latest commit. My fix was wrong also. The real fix is that we want to check the return code of wc not awk and echo 0 in that case. David Index: infrastructure/mk/bsd.port.mk =================================================================== RCS file: /cvs/ports/infrastructure/mk/bsd.port.mk,v retrieving revision 1.704 diff -u -p -r1.704 bsd.port.mk --- infrastructure/mk/bsd.port.mk 4 Jul 2005 12:32:51 -0000 1.704 +++ infrastructure/mk/bsd.port.mk 4 Jul 2005 15:49:58 -0000 @@ -2005,7 +2005,7 @@ ${_F}: else \ if grep -q "SIZE ($$file)" ${CHECKSUM_FILE}; then \ ${ECHO_MSG} ">> Size does not match for ${_F}"; \ - test `wc -c "$$file" 2>/dev/null|awk '{print $$1}' || echo 0` -lt 30000 && rm -f $$file; \ + test `wc -c "$$file" 2>/dev/null || echo 0 | awk '{print $$1}'` -lt 30000 && rm -f $$file; \ else \ ${ECHO_MSG} ">> No size recorded for ${_F}"; \ exit 0; \