On Tue, Nov 6, 2012 at 2:31 PM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank">brad.king@kitware.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 11/06/2012 08:22 AM, Petr Kmoch wrote:<br>
> I verified that the tests fail when CMAKE_MAKE_PROGRAM points to msbuild,<br>
> because 'cmake --build' does not operate on the .sln in such case.<br>
<br>
</div>FYI, a recent issue was filed that may require cmake --build to read<br>
the .sln file:<br>
<br>
 <a href="http://www.cmake.org/Bug/view.php?id=13623" target="_blank">http://www.cmake.org/Bug/view.php?id=13623</a><br>
<br>
It has some overlap with your work in case you want to try fixing it.<br>
<div class="im"><br>
> This means it makes no sense to run the tests when using msbuild.<br>
> Shall I send you a patch to disable the tests for it?<br>
> And if so, on what commit should I base it?<br>
<br>
</div>Please rebase all your commits on master and squash the not-on-msbuild<br>
part into the same commit that adds the test.  Include a comment<br>
explaining why it can't run with MSBuild.<br></blockquote><div><br>Yup, I'll do that.<br><br>Petr<br></div></div><br>