Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(GH-379) Use msbuild on unix when it is available #545

Merged
merged 1 commit into from
Jul 11, 2020

Conversation

AdmiringWorm
Copy link
Member

This pull request adds the ability to use msbuild on unix platforms when the executable is available.

With a fallback to xbuild if msbuild could not be found

resolves #379

@AdmiringWorm AdmiringWorm requested a review from gep13 July 9, 2020 21:32
@AdmiringWorm AdmiringWorm force-pushed the change-to-msbuild-unix branch 2 times, most recently from 73274d3 to 1b0aaa0 Compare July 10, 2020 18:50
@gep13
Copy link
Member

gep13 commented Jul 10, 2020

@AdmiringWorm are you still testing some things for this, or is it ready. In principle, I have no objections to this going in.

@AdmiringWorm
Copy link
Member Author

@gep13 I can't test it because it won't get past the GitVersion failures.

@AdmiringWorm
Copy link
Member Author

Running integration tests:
https://travis-ci.org/github/AdmiringWorm/Cake.Recipe.Tests/builds/707007054
https://ci.appveyor.com/project/AdmiringWorm/cake-recipe-tests/builds/34031219 (not important this one in this context).

Once this are all green (or at least until coveralls execution), then I will mark this one as ready.

This includes a fallback to xbuild if msbuild could not be found
@AdmiringWorm AdmiringWorm marked this pull request as ready for review July 11, 2020 11:49
@AdmiringWorm
Copy link
Member Author

@gep13 I believe this one is now ready for a review

Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gep13 gep13 merged commit 01dd22b into cake-contrib:develop Jul 11, 2020
@gep13
Copy link
Member

gep13 commented Jul 11, 2020

@AdmiringWorm thank you!

@AdmiringWorm AdmiringWorm deleted the change-to-msbuild-unix branch July 11, 2020 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use msbuild on unix when it is installed
2 participants