Merge pull request #2631 from ludovic-henry/fix-microsoftbuild
authormonojenkins <jo.shields+jenkins@xamarin.com>
Wed, 17 Feb 2016 17:25:30 +0000 (17:25 +0000)
committermonojenkins <jo.shields+jenkins@xamarin.com>
Wed, 17 Feb 2016 17:25:30 +0000 (17:25 +0000)
commit25b04d16d938df85b72479d1af667f8823fb6434
tree7922055f497b4a7b4c66bafcd145fcb941ffe3e4
parentd7da0cc294a4444e4632476463a5b11ef40978d7
parent0d0f435afb11441cfd8a6d570d5e7fd30f71fdfb
Merge pull request #2631 from ludovic-henry/fix-microsoftbuild

[Microsoft.Build] Fix Microsoft.Build.Execution.BuildManagerTest.BasicManualParallelBuilds

The previous implementation of ProcessWrapper would launch its own threads to read on the process stdout and stderr. We can more simply use OutputDataReceived and ErrorDataReceived which will do the same under the hood. We use as well the Exited event to track the end of the Process.