[llvm-commits] fix egrep options for Darwin in utils/llvmgrep

[Liang Wang]

On Fri, Jan 27, 2012 at 4:31 AM, Chris Lattner <clattner at apple.com> wrote:
> On Jan 24, 2012, at 6:01 PM, Liang Wang wrote:
>> Hi,
>>
>> Currently, llvmgrep shows only filename since it uses -l as egrep
>> option on Darwin.  It should use -H as for Linux.
>> The following patch fixes the problem.  OK for trunk?
>
> Sure, please commit.

Thanks.  I haven't got commit access yet.  Could you please help
commit this patch?

Thanks,
Liang.

>
> -Chris
>
>>
>> Thanks,
>> Liang.
>>
>>
>> Index: utils/llvmgrep
>> ===================================================================
>> --- utils/llvmgrep    (revision 148897)
>> +++ utils/llvmgrep    (working copy)
>> @@ -29,7 +29,7 @@
>>   cd $TOPDIR
>>   case `uname -s` in
>>     SunOS) grep_cmd="ggrep -H -n" ;;
>> -    Linux) grep_cmd="egrep -H -n" ;;
>> +    Linux|Darwin) grep_cmd="egrep -H -n" ;;
>>     *) grep_cmd="egrep -l -n" ;;
>>   esac
>>   ./utils/llvmdo -topdir "$TOPDIR" \
>> <egrep-options-for-darwin.patch>_______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>