Skip to content

Conversation

@wli3
Copy link

@wli3 wli3 commented Feb 11, 2019

for dotnet/sdk#2867 a manual insertion

@wli3 wli3 requested a review from a team February 11, 2019 00:29
@wli3
Copy link
Author

wli3 commented Feb 11, 2019

Also there was an insertion 10 hours ago, why it is so old?

@nguerrera
Copy link

nguerrera commented Feb 11, 2019

Looks like last insertion of toolset was 10 days ago, not 10 hours ago!

@nguerrera
Copy link

Build has been green a bunch since then. @chcosta @mmitche Thoughts?

Per dotnet/fsharp#5882
This file is no longer needed and removed

<Exec Command="find $(SdkOutputDirectory) -type d -exec chmod 755 {} \;" />
<Exec Command="find $(SdkOutputDirectory) -type f -exec chmod 644 {} \;" />
<Chmod Mode="755" Glob="$(SdkOutputDirectory)/FSharp/RunFsc.sh" />
Copy link
Author

Choose a reason for hiding this comment

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

@KevinRansom double check with you, we could remove this chmod right? per dotnet/fsharp#5882

Choose a reason for hiding this comment

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

@wli3 yes. I confirm removing this is fine and necessary.

@livarcocc
Copy link

@mmitche any chance we forgot to turn the subscription back on after Preview2 shipped?

@wli3 wli3 merged commit a8075c2 into dotnet:master Feb 11, 2019
@wli3 wli3 deleted the update-toolset-to-3.0.100-preview4.19110.1 branch February 11, 2019 19:08
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.

3 participants