-
Notifications
You must be signed in to change notification settings - Fork 181
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
Drop MSBuild, use dotnet build instead #1441
Drop MSBuild, use dotnet build instead #1441
Conversation
It's beautiful, @thomaslevesque. Thanks. Confession time: I didn't completely follow when last we talked about the build prerequisites. It seems that we now don't need VS or MSBuild, but you were concerned about some other extensions or plugins or something that I don't know what they are. Are you happy with |
I think we don't. But I'd like to check, first. I'll create a clean VM in Azure to make sure.
I don't remember what it might be.
No, I should update it. We might want to state clearly that it only works on Windows (because of ILMerge). It was implied by the VS requirement, but if that requirement is gone we should be more explicit. Currently it's a little understated:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
OK, so Visual Studio itself isn't required, but we still need
So I think the easiest way to describe the requirements is to say that we need VS2017 or at least the VS Build Tools. |
@@ -16,9 +16,7 @@ Ensure that the following are installed: | |||
|
|||
1. .NET 3.5, required by ILMerge | |||
|
|||
1. .NET framework 4.6.1 or later |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is implied by VS2017 (you can't have VS2017 without having net461 or later)
1d31f2b
to
6b291de
Compare
I do love me some deleted lines. Thanks, @thomaslevesque! |
Thanks for the merge! |
This change has been released as part of FakeItEasy 4.9.0. |
As discussed with @adamralph, not sure where/when it was.
We no longer need to call MSBuild explicitly, so we don't need VSWhere anymore.