Re: [PATCH (Apple Git) 01/13] .gitignore: Remove *.s as it matches *.S on case insensitive filesystem
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 30, 2019, 16:47 UTC
- Message-ID
- <xmqqy3723xei.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <20190129193818.8645-2-jeremyhu@apple.com>
Jeremy Huddleston Sequoia <jeremyhu@apple.com> writes:
> This was causing problems with ppc/sha1ppc.S
"causing problems"? How? The source file is already tracked so "git add" to record updates to it would work just fine, and "git status" would report "modified" for it, with or without "*.S" in the .gitignore file, no?
To emulate, I just added ".*S" to .gitignore and modified
ppc/sha1ppc.S to double check whta happens.We still have the build rule for %.s in our Makefile, so ignoring ".s" is still the right thing to do to make sure "git add dir/" won't add "dir/frotz.s" by accident. I sense the downside of doing so outweighs whatever benefit it would have on case incapable filesystems (and it is unclear what problem this change is trying to work around).
Show 18 quoted lines
> Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com> > --- > .gitignore | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/.gitignore b/.gitignore > index 0d77ea5894..a5db584576 100644 > --- a/.gitignore > +++ b/.gitignore > @@ -195,7 +195,7 @@ > *.deb > /git.spec > *.exe > -*.[aos] > +*.[ao] > *.py[co] > .depend/ > *.gcda