On 09/13/2011 11:22 AM, Brandon Casey wrote: > On 09/13/2011 11:05 AM, Jeff King wrote: >> On Tue, Sep 13, 2011 at 10:15:15AM -0500, Brandon Casey wrote: >> >>> ...and I see there is already an fnmatch_icase() in dir.c which adds >>> FNM_CASEFOLD when the global var ignore_case is set. So, maybe it's as >>> easy as: >>> [...] >>> - return (fnmatch(pattern, basename, 0) == 0); >>> + return (fnmatch_icase(pattern, basename, 0) == 0); >> >> OK, wow. That's exactly the level of easy I was hoping for. Do you want >> to roll that up into a patch with some tests? > > I haven't even tested that it runs. :) No, I was hoping someone > who was more interested would finish it, and maybe even test on > an affected system. Ok, I lied. Here's a series that needs testing by people on a case-insensitive filesystem and some comments. The first three patches are just housekeeping and can be accepted independently of the fourth patch which is marked WIP. The last patch implements the case-insensitive matching of attribute patterns, but I discovered that bad things can happen if git_config() is called more than once. Details are in the patch email. -Brandon [PATCH 1/4] attr.c: avoid inappropriate access to strbuf "buf" [PATCH 2/4] cleanup: use internal memory allocation wrapper [PATCH 3/4] builtin/mv.c: plug miniscule memory leak [PATCH 4/4] attr.c: respect core.ignorecase when matching attribute -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html