Re: [jamboost] [BBv2] warnings fix for VC6 (was Re: CVS and MSVC 6)

From: Vladimir Prus (ghost_at_[hidden])
Date: 2005-10-04 09:01:07

Hi Reece,

> Bojan Resnik wrote:
> > I fiddled some more with this and noticed that the actual command line
> > is:
> >
> > cl /Zm800 -nologo -TP /Z7 /Od /Ob0 /W4 /GR /MDd /Zc:forScope
> > /Zc:wchar_t /Wp64 /wd4675
> >
> > The "/Ze and /Za" error appeared because both /Zc options above are
> > not recognized by the compiler - they are specific to MSVC 7 and
> > newer. I put an "ECHO $(version) ;" in msvc.jam just above the
> > condition that enables these options, and it seems that the compilers
> > are correctly recognized as "6" and "7.1". I am not sure why the /Zc
> > flags are still added, though. The /Wp64 and /wd4675 should also be
> > reserved for MSVC newer than 6.
>
> The attached patch fixes both the /Wp64 command line error and /wd4675
> command line warning when using msvc-6.0.

Applied, thanks!

- Volodya

-- 
Vladimir Prus
http://vladimir_prus.blogspot.com
Boost.Build V2: http://boost.org/boost-build2