Re: [PATCH v5] Add an explicit GIT_DIR to the list of excludes

On Thu, Jun 5, 2014 at 3:15 AM, Pasha Bolokhov [off-list ref] wrote:

quoted

+       /* only add it if GIT_DIR does not end with '.git' or '/.git' */
+       if (len < 4 || strcmp(n_git + len - 4, ".git") ||
+           (len > 4 && n_git[len - 5] != '/')) {

Hmm.. should we exclude "foobar.git" as well?

    Why wouldn't we? Everything that has basename ".git" is hard-wired
to be excluded, but everything else, including "foobar.git" should be
added to the excludes manually... How is it better than just "foobar"?