Re: [PATCH] gitignore(5): Allow "foo/" in ignore list to match directory "foo"

Just to add my two eurocents: I think the patch is complicated enough that
we could go the other way round: while parsing the ignore entries, we can
plainly state that entries with a trailing slash are ignored:

-- snipsnap --
[PATCH] Warn if an ignore/exclude entry ends in a slash

Git does not like ignore entries ending in a slash; they will be ignored.
So just be honest and warn the user about it.

Signed-off-by: Johannes Schindelin <redacted>

---

 dir.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/dir.c b/dir.c
index 1b9cc7a..c3e9a0d 100644
--- a/dir.c
+++ b/dir.c

@@ -135,6 +135,11 @@ void add_exclude(const char *string, const char *base,
        }
        x->pattern = string;
        x->patternlen = strlen(string);
+       if (x->patternlen && x->pattern[x->patternlen - 1] == '/') {
+               warning("Ignoring ignore entry because of trailing slash: %s",
+                       string);