* 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; \

Reply via email to