r188833 - Look at lowercase version of argv[0] when determining driver mode

Hans Wennborg hans at chromium.org
Tue Mar 11 16:49:24 PDT 2014


On Tue, Mar 11, 2014 at 2:05 PM, Jordan Rose <jordan_rose at apple.com> wrote:
>
> On Mar 11, 2014, at 13:58, Dimitry Andric <dimitry at andric.com> wrote:
>
>> Hi Hans,
>>
>> I realize it's a little late to react on this commit. :-)
>>
>> But in any case, on FreeBSD we do have case sensitive filesystems, just like Linux and most other Unixes, and we are using a custom patch to detect "CC" as being an alias for "c++".  (This is for compatibility with a lot of existing build systems out there, which assume "CC" is the C++ compiler, and "cc" is the regular C compiler.)
>>
>> So I would appreciate it, if the toLowerCase transformation was surrounded with e.g. #ifdef _WIN32.  It is probably the only OS that has semi-random case for filenames?  (Or maybe OSX too, since that defaults to a case insensitive filesystem.)
>>
>> -Dimitry
>
> OS X is case-preserving enough that I wouldn't worry about 'CC' being used as 'cc'.

OK, I've addressed this in r203624.

Thanks,
Hans




More information about the cfe-commits mailing list