Use official flag for detailed symbols. Require VS2013 with update 3 or later to build.

[shuffle2]

No description provided.

[shuffle2]

/Zo is new to VS2013 with update 3, but requiring that compiler version or later also prevents users reporting bugs which have been fixed in VS updates (thus wasting our time).

[delroth]

Is there by any chance a way to get this version requirement into the
vcxproj file?
On Aug 10, 2014 2:52 PM, "shuffle2" notifications@github.com wrote:

/Zo is new to VS2013 with update 3, but requiring that compiler version or
later also prevents users reporting bugs which have been fixed in VS
updates (thus wasting our time).

—
Reply to this email directly or view it on GitHub
#777 (comment).

[shuffle2]

I spent ~5minutes and couldn't see an (easy) way to put a cl version check into msbuild, although I'm sure there is some way...

Making the precompiled headers better is still a thing I need to do...so hopefully soon it won't just be "random check in one of the many stdafx.h".

[shuffle2]

I guess it depends if you want to rely on msbuild version being tied to compiler version. If yes, it's easy...but I wouldn't want to rely on that.

[Tilka]

lgtm

[lioncash]

Looks good to me

[delroth]

LGTM too, please make sure our build documentation is up to date and
mentions (in some kind of FAQ section) what happens if you are building
with an old version and how to fix it (install this package, etc.)

Thanks!
On Aug 11, 2014 4:17 PM, "Lioncash" notifications@github.com wrote:

Looks good to me

—
Reply to this email directly or view it on GitHub
#777 (comment).

[lioncash]

Updated the necessary wiki links. shuffle2 could probably do the FAQ if necessary.