Skip to content

Upgrade dotnet cli to 2.1.503#2224

Closed
tmat wants to merge 1 commit intomasterfrom
dev/tmat/cli600
Closed

Upgrade dotnet cli to 2.1.503#2224
tmat wants to merge 1 commit intomasterfrom
dev/tmat/cli600

Conversation

@tmat
Copy link
Copy Markdown
Member

@tmat tmat commented Mar 12, 2019

This is needed to get new NuSpec licensing features. See #2003 for details.

@tmat
Copy link
Copy Markdown
Member Author

tmat commented Mar 12, 2019

@dagood Ok to update to 2.1.600?

@dagood
Copy link
Copy Markdown
Member

dagood commented Mar 12, 2019

Source-build doesn't have a v2.1.600-SDK tag, so as far as I know, can't do this update yet. @dseefeld @crummel @dleeapho in case I'm missing something. I'm not familiar with this SDK release in particular.

@dagood
Copy link
Copy Markdown
Member

dagood commented Mar 12, 2019

Wait, I missed that this is an Arcade master PR... why is 2.1.600 desired rather than a 3.0.100 preview?

@tmat
Copy link
Copy Markdown
Member Author

tmat commented Mar 12, 2019

We can't use 3.0 everywhere yet. Repos are not ready.

@tmat
Copy link
Copy Markdown
Member Author

tmat commented Mar 12, 2019

Would 2.1.503 work?

@dagood
Copy link
Copy Markdown
Member

dagood commented Mar 12, 2019

Yeah, there are tags for 2.1.503 and even 504 and 505 SDKs on https://github.com/dotnet/source-build.

@dagood
Copy link
Copy Markdown
Member

dagood commented Mar 12, 2019

And actually even more directly, source-build is using the 2.1.503 toolset itself: https://github.com/dotnet/source-build/blob/master/DotnetCLIVersion.txt

@tmat
Copy link
Copy Markdown
Member Author

tmat commented Mar 12, 2019

OK, I'll use 2.1.503

@tmat tmat changed the title Upgrade dotnet cli to 2.1.600 Upgrade dotnet cli to 2.1.503 Mar 12, 2019
@tmat tmat force-pushed the dev/tmat/cli600 branch from a81c3ae to afad274 Compare March 12, 2019 18:47
@tmat
Copy link
Copy Markdown
Member Author

tmat commented Mar 12, 2019

Hmm, actually I'll do this as part of #2003. On it's own it might break repos that do not suppress nuget licenseurl warning.

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.

2 participants