2010-09-09 Mihai Parparita <mihaip@chromium.org>
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Sep 2010 21:07:44 +0000 (21:07 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 9 Sep 2010 21:07:44 +0000 (21:07 +0000)
        Reviewed by Daniel Bates.

        svn-apply tries to delete directories it shouldn't
        https://bugs.webkit.org/show_bug.cgi?id=45424

        isDirectoryEmptyForRemoval had the wrong check. If an item in the
        directory is itself a directory, then the directory is definitely
        not empty.

        * Scripts/svn-apply:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@67115 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKitTools/ChangeLog
WebKitTools/Scripts/svn-apply

index 262127e..f680eb2 100644 (file)
@@ -1,3 +1,16 @@
+2010-09-09  Mihai Parparita  <mihaip@chromium.org>
+
+        Reviewed by Daniel Bates.
+
+        svn-apply tries to delete directories it shouldn't
+        https://bugs.webkit.org/show_bug.cgi?id=45424
+
+        isDirectoryEmptyForRemoval had the wrong check. If an item in the
+        directory is itself a directory, then the directory is definitely
+        not empty.
+
+        * Scripts/svn-apply:
+
 2010-09-09  Chris Fleizach  <cfleizach@apple.com>
 
         Fixing GTK and windows build failure.
index 33b2279..1cf9c01 100755 (executable)
@@ -280,7 +280,7 @@ sub isDirectoryEmptyForRemoval($)
     opendir DIR, $dir or die "Could not open '$dir' to list files: $?";
     for (my $item = readdir DIR; $item && $directoryIsEmpty; $item = readdir DIR) {
         next if exists $removeDirectoryIgnoreList{$item};
-        if (-d File::Spec->catdir($dir, $item)) {
+        if (-d File::Spec->catdir($dir, $item)) {
             $directoryIsEmpty = 0;
         } else {
             next if (scmWillDeleteFile(File::Spec->catdir($dir, $item)));