Subject: CVS commit: pkgsrc/mk/bulk
To: None <pkgsrc-changes@NetBSD.org>
From: Todd Vierling <tv@netbsd.org>
List: pkgsrc-changes
Date: 01/12/2005 22:51:54
Module Name:	pkgsrc
Committed By:	tv
Date:		Wed Jan 12 22:51:54 UTC 2005

Modified Files:
	pkgsrc/mk/bulk: bsd.bulk-pkg.mk

Log Message:
Fix checking of up to date binary packages.  The problem with DEPENDS
containing constructs of the form

    {perl>=5.8.3nb1,perl-thread>=5.8.3nb1}

is that a Bourne shell "for" interprets this as two separate items in the
list, nuking the {}s.  The above will never succeed and thus always cause
Perl-dependent packages to be rebuilt unconditionally.

Just adding more \s isn't enough -- the parsing of the depends list has to
be moved out of the "for" clause.  So, echo them one per line and use "read"
to pull them in without allowing the shell to peek at those characters.


To generate a diff of this commit:
cvs rdiff -r1.67 -r1.68 pkgsrc/mk/bulk/bsd.bulk-pkg.mk

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.