Re: [PATCH] Ignore boilerplate logo from MSVC on stderr.

[Peter Rosin]

Den 2010-08-16 21:26 skrev Peter Rosin:
> Den 2010-08-16 20:45 skrev Ralf Wildenhues:
>> Hi Peter,
>>
>> * Peter Rosin wrote on Mon, Aug 16, 2010 at 08:14:50PM CEST:
>>> The logo of my version of MSVC is:
>>>
>>> ----8<----
>>> Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 14.00.50727.762 for 
>>> 80x86
>>> Copyright (C) Microsoft Corporation.  All rights reserved.
>>>
>>> ----8<----
>>
>> Without having looked at the patch or your post any further at all:
>> what about other versions of this compiler?  What about localized
>> versions of it?
> 
> Good catch, I knew they were very similar and assumed my matching was just
> about wide enough. But since I didn't check, Murphy made sure that it wasn't
> working for everybody, see below...
> 
> Other versions:
> MSVC 6 (I don't have access to anything older than that)
> ----8<----
> Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 12.00.8804 for 80x86
> Copyright (C) Microsoft Corp 1984-1998. All rights reserved.
> 
> ----8<----
> 
> MSVC++ Toolkit 2003
> ----8<----
> Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 13.10.3077 for 80x86
> Copyright (C) Microsoft Corporation 1984-2002. All rights reserved.
> 
> ----8<----
> 
> MSVC 2008 (snipped from Internet, not 100% certain)
> ----8<----
> Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 15.00.30729.01 for 
> 80x86
> Copyright (C) Microsoft Corporation.  All rights reserved.
> 
> ----8<----
> 
> MSVC 2010 (snipped from Internet, not 100% certain)
> ----8<----
> Microsoft (R) 32-bit C/C++ Optimizing Compiler Version 16.00 for 80x86
> Copyright (C) Microsoft Corporation.  All rights reserved.
> 
> ----8<----
> 
> Don't know about localization, sorry... Someone else?
> 
> So, I've updated to also match MSVC 6 (Corp instead of Corporation).


Ping.

We have a proposed patch that fixes problems.  We have a vaporware
competing patch that is supposed to be nicer but only fixes part of
the problem.

It should be very easy to revert this proposed patch if/when the vapor
materializes (and if we want to).

Cheers,
Peter