Title: [89196] trunk/Tools
Revision
89196
Author
[email protected]
Date
2011-06-18 12:33:28 -0700 (Sat, 18 Jun 2011)

Log Message

2011-06-18  Berend-Jan Wever  <[email protected]>

        Reviewed by Eric Seidel.

        Some names with spaces are incorrectly reported as not containing spaces
        while running prepare-ChangeLog
        https://bugs.webkit.org/show_bug.cgi?id=55572

        * Scripts/VCSUtils.pm: Updated CHANGE_LOG_NAME check for spaces to allow
          non-alphanumeric characters in names.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (89195 => 89196)


--- trunk/Tools/ChangeLog	2011-06-18 19:24:47 UTC (rev 89195)
+++ trunk/Tools/ChangeLog	2011-06-18 19:33:28 UTC (rev 89196)
@@ -1,3 +1,14 @@
+2011-06-18  Berend-Jan Wever  <[email protected]>
+
+        Reviewed by Eric Seidel.
+
+        Some names with spaces are incorrectly reported as not containing spaces
+        while running prepare-ChangeLog
+        https://bugs.webkit.org/show_bug.cgi?id=55572
+
+        * Scripts/VCSUtils.pm: Updated CHANGE_LOG_NAME check for spaces to allow
+          non-alphanumeric characters in names.
+
 2011-06-18  Michael Saboff  <[email protected]>
 
         Reviewed by Eric Seidel.

Modified: trunk/Tools/Scripts/VCSUtils.pm (89195 => 89196)


--- trunk/Tools/Scripts/VCSUtils.pm	2011-06-18 19:24:47 UTC (rev 89195)
+++ trunk/Tools/Scripts/VCSUtils.pm	2011-06-18 19:33:28 UTC (rev 89196)
@@ -1765,7 +1765,7 @@
 
     changeLogNameError("Failed to determine ChangeLog name.") unless $name;
     # getpwuid seems to always succeed on windows, returning the username instead of the full name.  This check will catch that case.
-    changeLogNameError("'$name' does not contain a space!  ChangeLogs should contain your full name.") unless ($name =~ /\w \w/);
+    changeLogNameError("'$name' does not contain a space!  ChangeLogs should contain your full name.") unless ($name =~ /\S\s\S/);
 
     return $name;
 }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to