Opened 5 years ago

Closed 5 years ago

#208 closed defect (fixed)

grep --include and --exclude options are not case-insensitve

Reported by: Steven Levine Owned by:
Priority: minor Milestone:
Component: *none Version:
Severity: medium Keywords:
Cc:

Description

grep -r --include=*.c foo will not check files named *.C because the fnmatch() call is case-sensitive

To change this behavior, modify:

lib\exclude.c:384

bool matched = ((*matcher) (pattern, f, options) == 0);

to

bool matched = ((*matcher) (pattern, f, options | FNM_CASEFOLD) == 0);

Change History (3)

comment:1 Changed 5 years ago by Silvan Scherrer

fixed in the soon to be released grep. please test when v3.3 is available as rpm in exp.

comment:2 Changed 5 years ago by Silvan Scherrer

Priority: minorFeedback pending

please test, as v3.3 with the fix is up since a couple of days. Further grep issues please report at https://github.com/bitwiseworks/grep-os2

comment:3 Changed 5 years ago by Steven Levine

Priority: Feedback pendingminor
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.