Skip to content
This repository has been archived by the owner on Apr 20, 2023. It is now read-only.

Update dependencies #6135

Merged
merged 3 commits into from
Mar 24, 2017
Merged

Update dependencies #6135

merged 3 commits into from
Mar 24, 2017

Conversation

ellismg
Copy link

@ellismg ellismg commented Mar 24, 2017

  • Pull dotnet-bot's PR.
  • Fixup OSX RID.

dotnet-bot and others added 2 commits March 23, 2017 23:58
Since Microsoft.NETCore.App 2.0 will only support 10.12 and later, the
RID in the runtime package has changed to osx.10.12. We need to update
so we pick up crossgen from the right place.
@ellismg
Copy link
Author

ellismg commented Mar 24, 2017

@eerhardt @livarcocc

Going forward, Microsoft.NETCore.App only supports OSX 10.12 and newer.

@ellismg
Copy link
Author

ellismg commented Mar 24, 2017

test Debian8.2 x64 Debug Build please
test Fedora24 x64 Release Build please
test OSX x64 Release Build please

@ellismg
Copy link
Author

ellismg commented Mar 24, 2017

OSX CI run is failing because it doesn't run on 10.12. I can fix this in CI but I'm guessing we'all need to make a similar change in the official build system.

@ellismg
Copy link
Author

ellismg commented Mar 24, 2017

Needs #6137 to be merged and the OSX job to be regenerated. @livarcocc or @eerhardt can you help me move the official build machines to a 10.12 pool (or verify that the pool we are using is already a 10.12 pool?)

@livarcocc livarcocc modified the milestone: 2.0.0 Mar 24, 2017
@eerhardt
Copy link
Member

https://devdiv.visualstudio.com/DevDiv/_build/definitionEditor?definitionId=5445&_a=general

Looks like it just says:

image

Should be able to update this to tell it we need 10.12 (however you do that in VSTS).

@livarcocc
Copy link

@dotnet-bot Test OSX x64 Release Build

@livarcocc
Copy link

@dotnet-bot Test this please.

@livarcocc livarcocc merged commit 47158e7 into dotnet:master Mar 24, 2017
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.

6 participants