[DOC] Default value of IgnoreStandardErrorWarningFormat inconsistent between product & MSDN

[TheRealPiotrP]

According to MSDN, IgnoreStandardErrorWarningFormat should default to true. This shows that it's false. They should be consistent.

[shift-evgeny]

This isn't just an inconsistency, it's a horrible, horrible hack to fail the build due to some "standard error and warning format" messages. This so-called "standard format" isn't even documented on the linked MSDN page and this is certainly unexpected behaviour for someone creating a pre-build or post-build step in Visual Studio. This hack should certainly be off by default, like the documentation says it is and if someone does want to turn it on, they should have documentation on exactly what strings it looks for.

Looks like I'm not the only one to run into it: http://stackoverflow.com/questions/39466090/why-outputting-error-in-post-build-event-breaks-my-build-in-vs2012

[jeffkl]

The property also says it defaults to true: https://github.com/Microsoft/msbuild/blob/56a7b356b96c4d2ddbde7d268990b62a7f3bfca8/src/XMakeTasks/Exec.cs#L106

From what I can tell, it's always been false by default so we'll need to determine what impact there would be, if any, if we change it.

[Sarabeth-Jaffe-Microsoft]

Thanks for the heads up on this. Looking into it now.

[rainersigwald]

I don't think this closes the issue. We need the docs site updated, too.

[Sarabeth-Jaffe-Microsoft]

MSDN docs has been updated to correct default value of false.

This exact page hasn't been moved over to docs, so I added a line specifying the default value in docs under "IgnoreStandardErrorWarningFormat": https://docs.microsoft.com/en-us/visualstudio/msbuild/exec-task