{"thread":{"id":"4415","subject":"[PATCH] builtin-grep: pass ignore case option to external grep","startedAt":"2006-06-06T23:15:16Z","lastAt":"2006-06-06T23:27:02Z","messageCount":2,"participants":["Robert Fitzsimons","Linus Torvalds"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"21354","messageId":"20060606231516.GA18405@localhost","threadId":"4415","inReplyTo":null,"subject":"[PATCH] builtin-grep: pass ignore case option to external grep","fromName":"Robert Fitzsimons","fromEmail":"robfitz@273k.net","sentAt":"2006-06-06T23:15:16Z","receivedAt":"2006-06-06T23:15:16Z","isPatch":true,"sender":{"key":"robfitz@273k.net","avatar":null},"body":"Don't just read the --ignore-case/-i option, pass the flag on to the\nexternal grep program.\n\nSigned-off-by: Robert Fitzsimons <robfitz@273k.net>\n---\n builtin-grep.c |    2 ++\n 1 files changed, 2 insertions(+), 0 deletions(-)\n\ndiff --git a/builtin-grep.c b/builtin-grep.c\nindex acc4eea..5fac570 100644\n--- a/builtin-grep.c\n+++ b/builtin-grep.c\n@@ -459,6 +459,8 @@ static int external_grep(struct grep_opt\n \t\tpush_arg(\"-n\");\n \tif (opt->regflags & REG_EXTENDED)\n \t\tpush_arg(\"-E\");\n+\tif (opt->regflags & REG_ICASE)\n+\t\tpush_arg(\"-i\");\n \tif (opt->word_regexp)\n \t\tpush_arg(\"-w\");\n \tif (opt->name_only)\n-- \n1.3.3.g16a4-dirty\n"},{"id":"21355","messageId":"Pine.LNX.4.64.0606061625430.5498@g5.osdl.org","threadId":"4415","inReplyTo":"20060606231516.GA18405@localhost","subject":"Re: [PATCH] builtin-grep: pass ignore case option to external grep","fromName":"Linus Torvalds","fromEmail":"torvalds@osdl.org","sentAt":"2006-06-06T23:27:02Z","receivedAt":"2006-06-06T23:27:02Z","isPatch":true,"sender":{"key":"torvalds@linux-foundation.org","avatar":"https://avatars.githubusercontent.com/u/1024025?v=4"},"body":"\n\nOn Tue, 6 Jun 2006, Robert Fitzsimons wrote:\n>\n> Don't just read the --ignore-case/-i option, pass the flag on to the\n> external grep program.\n\nOops. How did we miss that one ;)\n\nEmbarrassing.\n\n\t\tLinus\n"}]}