ports/102100: [PATCH] bsd.port.mk: Add some quotes in 'missing' target (improve handling of duplicate origins)
Shaun Amott
shaun at FreeBSD.org
Tue Aug 15 16:10:28 UTC 2006
>Number: 102100
>Category: ports
>Synopsis: [PATCH] bsd.port.mk: Add some quotes in 'missing' target (improve handling of duplicate origins)
>Confidential: no
>Severity: non-critical
>Priority: low
>Responsible: freebsd-ports-bugs
>State: open
>Quarter:
>Keywords:
>Date-Required:
>Class: sw-bug
>Submitter-Id: current-users
>Arrival-Date: Tue Aug 15 16:10:20 GMT 2006
>Closed-Date:
>Last-Modified:
>Originator: Shaun Amott
>Release: FreeBSD 6.1-STABLE i386
>Organization:
>Environment:
>Description:
In the event there are multiple ORIGINs recorded for a particular
dependency, 'make missing' will spit out an error. This is non-critical,
and an unlikely scenario, but the handling of this situation could be
improved by adding some quotes, as below.
>How-To-Repeat:
charon# pkg_info -qO devel/libtool15
libtool-1.5.22_1
libtool-1.5.22_2
charon# make missing
lang/php5
devel/php5-pcre
textproc/php5-xml
[: libtool-1.5.22_1: unexpected operator
>Fix:
--- bsd.port.mk.diff begins here ---
Index: bsd.port.mk
===================================================================
RCS file: /home/pcvs/ports/Mk/bsd.port.mk,v
retrieving revision 1.540
diff -u -r1.540 bsd.port.mk
--- bsd.port.mk 14 Aug 2006 13:24:18 -0000 1.540
+++ bsd.port.mk 15 Aug 2006 16:00:46 -0000
@@ -5190,7 +5190,7 @@
@for dir in $$(${ALL-DEPENDS-LIST}); do \
THISORIGIN=$$(${ECHO_CMD} $$dir | ${SED} 's,${PORTSDIR}/,,'); \
installed=$$(${PKG_INFO} -qO $${THISORIGIN}); \
- if [ -z $$installed ]; then \
+ if [ -z "$$installed" ]; then \
${ECHO_CMD} $$THISORIGIN; \
fi \
done
--- bsd.port.mk.diff ends here ---
>Release-Note:
>Audit-Trail:
>Unformatted:
More information about the freebsd-ports-bugs
mailing list