Subject: CVS commit: pkgsrc/mk/bulk
To: None <pkgsrc-changes@NetBSD.org>
From: Roland Illig <rillig@netbsd.org>
List: pkgsrc-changes
Date: 11/10/2005 07:04:00
Module Name:	pkgsrc
Committed By:	rillig
Date:		Thu Nov 10 07:04:00 UTC 2005

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

Log Message:
Don't rely on the INDEXFILE having trailing white-space in each line.
Whoever had written such unusual code should have left a note in the
printindex script that the trailing white-space was required. It is such
an uncommon assumption that I treated it as a typo and removed it. Now
it's fixed. I also replaced the use of grep|awk with a simple awk, as
grep would have interpreted regular expressions in the package name.

In general, please think more about _appropriate_ ways to handle data.
grep(1) is not a tool for searching strings. It's a tool for searching
regular expressions.


To generate a diff of this commit:
cvs rdiff -r1.91 -r1.92 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.