Skip to content

GH2519: Changes for stable Visual Studio 2019 release#2520

Merged
devlead merged 1 commit intocake-build:developfrom
Roemer:feature/GH-2519-VS2019_support
Apr 15, 2019
Merged

GH2519: Changes for stable Visual Studio 2019 release#2520
devlead merged 1 commit intocake-build:developfrom
Roemer:feature/GH-2519-VS2019_support

Conversation

@Roemer
Copy link
Copy Markdown
Contributor

@Roemer Roemer commented Apr 4, 2019

Changes regarding the release of Visual Studio 2019.
Fixes #2519

"/Program86/Microsoft Visual Studio/2017/Enterprise/MSBuild/15.0/Bin/amd64/MSBuild.exe",
"/Program86/Microsoft Visual Studio/2019/Preview/MSBuild/Current/Bin/MSBuild.exe",
"/Program86/Microsoft Visual Studio/2019/Preview/MSBuild/Current/Bin/amd64/MSBuild.exe",
"/Program86/Microsoft Visual Studio/2019/Professional/MSBuild/Current/Bin/MSBuild.exe",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there a reason that you chose Professional here, rather than Enterprise, similar to what was done for 2017?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

As I only have the Professional version, I can verify only that path.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

But if you want, for consistency reasons, I can us the same as it is fairly obvious that it should be correct.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

There is no "need" for them to be consistent, I was just curious.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think it would be good if we added tests for the other editions as well.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@patriksvensson Should I have a look at that (that would probably mean making the MSBuildRunnerFixture configurable (to "fake" different VS installation"). Or should that be done probably later as I think cake should have VS2019 support ready asap.

@Roemer
Copy link
Copy Markdown
Contributor Author

Roemer commented Apr 4, 2019

For some reason, I cannot access the bitrise ci to check why it failed.

@gep13
Copy link
Copy Markdown
Member

gep13 commented Apr 4, 2019

@Roemer said...
For some reason, I cannot access the bitrise ci to check why it failed.

Unfortunately, the bitrise builds can't be made public :-(

Looking at the log, this is nothing to do with your change, so don't worry about it. Something we need to fix.

@devlead
Copy link
Copy Markdown
Member

devlead commented Apr 4, 2019

Unfortunately, the bitrise builds can't be made public :-(

Actually they now can, but they need to be recreated, I've recreated the Ubuntu Bitrise build and disabled old build, changed which is required for PRs to the new.

Issue was an expired ssh key, new build uses https which is what they allow for public projects.

@devlead
Copy link
Copy Markdown
Member

devlead commented Apr 4, 2019

Bitrise macOS build now recreated and public. Old OSX disabled.

@Roemer
Copy link
Copy Markdown
Contributor Author

Roemer commented Apr 4, 2019

So how do we go on from here? Are any more changes needed?

Copy link
Copy Markdown
Member

@devlead devlead left a comment

Choose a reason for hiding this comment

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

LGTM 👍
@Roemer the additonal tests can be sent as a follow up PR.

@devlead devlead merged commit 18d66b0 into cake-build:develop Apr 15, 2019
@devlead
Copy link
Copy Markdown
Member

devlead commented Apr 15, 2019

@Roemer your changes have been merged, thanks for your contribution 👍

@realrubberduckdev
Copy link
Copy Markdown

@devlead Hi, do you know when will it be in a release?
(Or is there a better way for me to find that out without asking here?)

@devlead
Copy link
Copy Markdown
Member

devlead commented Apr 16, 2019

It'll be in a release as we're happy with what's in the current milestone
https://github.com/cake-build/cake/milestone/59

Then we'll announce it before release at
https://twitter.com/cakebuildnet
https://gitter.im/cake-build/cake
https://gitter.im/cake-contrib/Lobby

A realease will appear at
https://github.com/cake-build/cake/releases

new packages here:
https://www.nuget.org/profiles/cake-build

A tweet will notify when package is out at
https://twitter.com/cakebuildnet
https://gitter.im/cake-build/cake
https://gitter.im/cake-contrib/Lobby

A blog post here:
https://cakebuild.net/blog/Release-Notes/

We'll also post to medium and linked in and a few other places

There's a recent YouTube video showing the process when Gary live streamed the release of 0.33.0 available at
https://youtu.be/cHtrJAc7sAQ

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.

Not able to build project with ToolsVersion="15.0"

5 participants