[PATCH 2 of 4] Win32: handling of localized MSVC cl output

Maxim Dounin mdounin at mdounin.ru
Fri Feb 10 04:33:01 UTC 2023


Hello!

On Thu, Feb 09, 2023 at 06:55:22AM +0000, Pavel Pautov via nginx-devel wrote:

> The 'for 80x86' part of cl version (in auto/configure output) 
> always felt odd. Given it's used in subsequent patch, can we at 
> least strip localized 'for' from NGX_MSVC_VER? Having it in 
> otherwise non-localized output of "nginx -V" doesn't seem right 
> either.

The "nginx -V" output simply shows how the compiler identifies 
itself.  The patch has no intention to change this, it just 
ensures that output of localized MSVC cl versions is recognized by 
nginx.  And, actually, I would rather object stripping the 
details, since any aspects of compiler identification might be 
important.

-- 
Maxim Dounin
http://mdounin.ru/


More information about the nginx-devel mailing list