diff options
author | rillig <rillig> | 2005-08-06 22:41:07 +0000 |
---|---|---|
committer | rillig <rillig> | 2005-08-06 22:41:07 +0000 |
commit | 28a00184895629bca63eaa5dd19e7b38260cbf45 (patch) | |
tree | 2a68ddfa160761016ccd5abd5c6355949884a736 | |
parent | f7c7ead263624e560434100f246a3497ac4e7da5 (diff) | |
download | pkgsrc-28a00184895629bca63eaa5dd19e7b38260cbf45.tar.gz |
Fixed a bug which led to an endless loop. While happily changing the
code, I had forgotten one assumption I made. It was that in the ifelse
chain the case ( && ) couldn't occur because
the outer loop already cought that. And then I had inserted code that
violated exactly this assumption. Fixed.
-rw-r--r-- | pkgtools/pkglint/files/pkglint.pl | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/pkgtools/pkglint/files/pkglint.pl b/pkgtools/pkglint/files/pkglint.pl index 58b927bd20c..989eee678c0 100644 --- a/pkgtools/pkglint/files/pkglint.pl +++ b/pkgtools/pkglint/files/pkglint.pl @@ -11,7 +11,7 @@ # Freely redistributable. Absolutely no warranty. # # From Id: portlint.pl,v 1.64 1998/02/28 02:34:05 itojun Exp -# $NetBSD: pkglint.pl,v 1.234 2005/08/06 22:24:07 rillig Exp $ +# $NetBSD: pkglint.pl,v 1.235 2005/08/06 22:41:07 rillig Exp $ # # This version contains lots of changes necessary for NetBSD packages # done by: @@ -2108,7 +2108,6 @@ sub check_category($) { } elsif ($is_wip && $line->text eq "") { # ignore the special case "wip", which defines its own "index" target. - $lines_atend = true; } else { $line->log_error("SUBDIR+= line expected."); @@ -2133,7 +2132,7 @@ sub check_category($) { $fetch_next_line = true; } - if ($lines_index == $#{$lines} - 1) { + if ($lines_index == $#{$lines} - 1 || ($is_wip && $line->text eq "")) { $lines_atend = true; } $filesys_atend = ($filesys_index == @filesys_subdirs); |