-for dir in `grep "^[^# ][^ ]* *[^ ][^ ]* *[^ ][^ ]*$" packages | sed "s/ .*//"`
-do
- # We would like to just check for an _darcs directory here, but in
- # an lndir tree we avoid making _darcs directories, so it doesn't
- # exist. We therefore require that every repo has a LICENSE file
- # instead.
- if test ! -f $dir/LICENSE
- then
- echo "Error: $dir/LICENSE doesn't exist." >&2
- echo "Maybe you haven't done './darcs-all get'?" >&2
- exit 1
- fi
-done
+open PACKAGES, "< packages";
+while (<PACKAGES>) {
+ if (/^#/) {
+ # Comment; do nothing
+ }
+ elsif (/^([a-zA-Z0-9\/.-]+) *[^ ]+ *[^ ]+$/) {
+ $dir = $1;
+
+ # We would like to just check for an _darcs directory here, but in
+ # an lndir tree we avoid making _darcs directories, so it doesn't
+ # exist. We therefore require that every repo has a LICENSE file
+ # instead.
+ if (! -f "$dir/LICENSE") {
+ print STDERR "Error: $dir/LICENSE doesn't exist.\n";
+ die "Maybe you haven't done './darcs-all get'?";
+ }
+ }
+ elsif (/^([a-zA-Z0-9\/.-]+) *[^ ]+ *[^ ]+ *[^ ]+$/) {
+ # These are lines which refer to optional repositories, so their
+ # absence isn't an error.
+ }
+ else {
+ die "Bad line in packages file: $_";
+ }
+}
+close PACKAGES;