Permalink
Browse files

Remove a bunch of extraneous ** from patterns in etc/gitignore

These were added in ef3e533, but they
are hurting, not helping. Consider the following patterns:

foo/
**/foo/
foo/**

The first will match all directories named "foo" (and anything inside
them) no matter where they reside in the repository.

The second will match directories named "foo" (and anything inside
them), but only if they are below the .gitignore in the directory
structure.

The third will match anything inside all directories named "foo".

Because Git only tracks files, not directories, the first and third
patterns are essentially equivalent. Since the first is simpler, we
should prefer it.

The second pattern can be useful in some cases, but as far as I can tell
it is not actually desired in any of the cases where we are currently
using it.

So, I've replaced all instances of patterns 2 and 3 above with pattern
1.
  • Loading branch information...
1 parent d0afcf3 commit efcb0fe8140c04b004b768f8dd3812543f12bdb7 @aroben aroben committed Jun 11, 2012
Showing with 6 additions and 8 deletions.
  1. +6 −8 etc/gitignore
View
@@ -1,14 +1,12 @@
-
#################
## Eclipse
#################
*.pydevproject
.project
.metadata
-bin/**
-tmp/**
-tmp/**/*
+bin/
+tmp/
*.tmp
*.bak
*.swp
@@ -44,8 +42,8 @@ local.properties
*.sln.docstates
# Build results
-**/[Dd]ebug/
-**/[Rr]elease/
+[Dd]ebug/
+[Rr]elease/
*_i.c
*_p.c
*.ilk
@@ -63,10 +61,10 @@ local.properties
*.tmp
*.vspscc
.builds
-**/*.dotCover
+*.dotCover
## TODO: If you have NuGet Package Restore enabled, uncomment this
-#**/packages/
+#packages/
# Visual C++ cache files
ipch/

0 comments on commit efcb0fe

Please sign in to comment.