X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=Porting%2Fcheck83.pl;h=5851d9ffb63246cb1a5b288fcbf8445265bbbd15;hb=8bbf3450a1ff0a3996dade29a4194cc0939d871f;hp=0522bc0d3367dc85355f669092aab49d900a77be;hpb=4d8cc5f8cae495ff0bcad4a9e629ad4bab92c057;p=p5sagit%2Fp5-mst-13.2.git

diff --git a/Porting/check83.pl b/Porting/check83.pl
index 0522bc0..5851d9f 100644
--- a/Porting/check83.pl
+++ b/Porting/check83.pl
@@ -1,8 +1,20 @@
-#!/usr/local/bin/perl
+#!/usr/bin/perl
 
 # Check whether there are naming conflicts when names are truncated to
 # the DOSish case-ignoring 8.3 format, plus other portability no-nos.
 
+# The "8.3 rule" is loose: "if reducing the directory entry names
+# within one directory to lowercase and 8.3-truncated causes
+# conflicts, that's a bad thing".  So the rule is NOT the strict
+# "no filename shall be longer than eight and a suffix if present
+# not longer than three".
+
+# TODO: this doesn't actually check for *directory entries*, what this
+# does is to check for *MANIFEST entries*, which are only files, not
+# directories.  In other words, a 8.3 conflict between a directory
+# "abcdefghx" and a file "abcdefghy" wouldn't be noticed-- or even for
+# a directory "abcdefgh" and a file "abcdefghy".
+
 sub eight_dot_three {
     my ($dir, $base, $ext) = ($_[0] =~ m!^(?:(.+)/)?([^/.]+)(?:\.([^/.]+))?$!);
     my $file = $base . defined $ext ? ".$ext" : "";
@@ -15,7 +27,7 @@ sub eight_dot_three {
 	warn "$file: filename contains non-portable characters\n";
     }
     if (length $file > 30) {
-	warn "$file: filename longer than 30 characters\n";
+	warn "$file: filename longer than 30 characters\n"; # make up a limit
     }
     if (defined $dir) {
 	return ($dir, defined $ext ? "$dir/$base.$ext" : "$dir/$base");