[Please keep everyone cc'd in the conversation, especially the mailing
list. I added it back for now.
Also please don't top post on the list]
On 02/25, Olga Pshenichnikova wrote:
> No, it isn't empty, but I found the problem.
> Problem was that I handled subdirectories structure in exclude file:
>
> design/dir1
> design/dir2
>
> But
>
> design/dir3 wasn't ignored and wasn't controlled.
>
> So, my problem take place when some sub directory both isn't ignored AND
> isn't controlled even if it isn't empty.
I think that's a bug, I think this directory should only be removed if
-x is given. I haven't come up with a patch yet, but here's a test
demonstrating the failure.
diff --git a/t/t7300-clean.sh b/t/t7300-clean.sh
index 86ceb38..0961007 100755
--- a/t/t7300-clean.sh
+++ b/t/t7300-clean.sh
@@ -653,4 +653,13 @@ test_expect_success 'git clean -d respects pathspecs
(pathspec is prefix of dir)
test_path_is_dir foobar
'
+test_expect_failure 'git clean -d does not clean ignored files in subdir' '
+ mkdir -p sub/dir &&
+ >sub/dir/file &&
+ test_when_finished rm .gitignore &&
+ echo sub/dir/ >.gitignore &&
+ git clean -df sub &&
+ test_path_is_file sub/dir/file
+'
+
test_done
> Thank you for response!
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
