Skip to content
This repository was archived by the owner on Jan 11, 2024. It is now read-only.

Use the built tasks to update dotnet/versions - #1935

Merged
dagood merged 1 commit into
dotnet:masterfrom
dagood:local-versions-updater/master
Feb 28, 2018
Merged

Use the built tasks to update dotnet/versions#1935
dagood merged 1 commit into
dotnet:masterfrom
dagood:local-versions-updater/master

Conversation

@dagood

@dagood dagood commented Feb 28, 2018

Copy link
Copy Markdown
Member

This allows us to use the TLS 1.2 UpdatePublishedVersions fix without upgrading the BuildTools toolset version.

Doing this properly (updating the BuildTools toolset version) is tracked by #1934. I hit enough obstacles that I don't think it's in scope for First Responder.

https://github.com/dotnet/core-eng/issues/2734: Unblocks auto-update of master BuildTools builds to subscribing repos. This change will need to be ported to release/2.1, at least.

This allows us to use the TLS 1.2 UpdatePublishedVersions fix without upgrading the BuildTools toolset version.
@dagood
dagood requested review from MattGal and weshaggard February 28, 2018 03:35
Comment thread build.proj

TODO: Remove this workaround once BuildTools uses a new BuildTools version: https://github.com/dotnet/buildtools/issues/1934
-->
<UsingTask TaskName="UpdatePublishedVersions"

This comment was marked as spam.

This comment was marked as spam.

@MattGal MattGal left a comment

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.

LGTM

@dagood
dagood merged commit bdffc61 into dotnet:master Feb 28, 2018
@dagood
dagood deleted the local-versions-updater/master branch February 28, 2018 17:48
@dagood

dagood commented Feb 28, 2018

Copy link
Copy Markdown
Member Author

Didn't work in the official build: https://devdiv.visualstudio.com/DevDiv/_build/index?buildId=1427170. Looking into it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants