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

Update CoreClr, CoreFx, External, ProjectNTfs to beta-24515-02, beta-devapi-24515-01, beta-24514-00, beta-24515-00, respectively (master)#11706

Closed
dotnet-bot wants to merge 1 commit into
dotnet:masterfrom
dotnet-bot:master-UpdateDependencies
Closed

Update CoreClr, CoreFx, External, ProjectNTfs to beta-24515-02, beta-devapi-24515-01, beta-24514-00, beta-24515-00, respectively (master)#11706
dotnet-bot wants to merge 1 commit into
dotnet:masterfrom
dotnet-bot:master-UpdateDependencies

Conversation

@dotnet-bot
Copy link
Copy Markdown

@dotnet-bot dotnet-bot commented Sep 14, 2016

/cc @dotnet/corefx-contrib

@stephentoub
Copy link
Copy Markdown
Member

@adityamandaleeka, another seg fault on CentOS. Do you want me to flag all of these for you I see, or do you have it covered via other means?

@dotnet-bot dotnet-bot changed the title Update CoreClr, CoreFx, External, ProjectNTfs to beta-24514-02, beta-devapi-24514-01, beta-24514-00, beta-24514-00, respectively (master) Update CoreClr, CoreFx, External, ProjectNTfs to beta-24514-03, beta-devapi-24514-01, beta-24514-00, beta-24514-00, respectively (master) Sep 14, 2016
@dotnet-bot dotnet-bot force-pushed the master-UpdateDependencies branch from 75b97d8 to 0bd0677 Compare September 14, 2016 16:13
@joperezr
Copy link
Copy Markdown
Member

I'll take care of this update.

@joperezr joperezr self-assigned this Sep 14, 2016
@tarekgh
Copy link
Copy Markdown
Member

tarekgh commented Sep 14, 2016

CC myself @tarekgh to get the news about this one.

@dotnet-bot dotnet-bot force-pushed the master-UpdateDependencies branch from 0bd0677 to 5b95c65 Compare September 14, 2016 23:52
@dotnet-bot dotnet-bot changed the title Update CoreClr, CoreFx, External, ProjectNTfs to beta-24514-03, beta-devapi-24514-01, beta-24514-00, beta-24514-00, respectively (master) Update CoreClr, CoreFx, External, ProjectNTfs to beta-24515-02, beta-devapi-24515-01, beta-24514-00, beta-24515-00, respectively (master) Sep 15, 2016
@dotnet-bot dotnet-bot force-pushed the master-UpdateDependencies branch from 5b95c65 to 2c982f0 Compare September 15, 2016 08:19
…devapi-24515-01, beta-24514-00, beta-24515-00, respectively
@joperezr
Copy link
Copy Markdown
Member

CI is failing due to a bug I introduced to coreclr String class. I'm fixing that with PR dotnet/coreclr#7212. Once that is merged and a new coreclr package is pushed, I'll fix this PR and merge it.

@joperezr joperezr force-pushed the master-UpdateDependencies branch from 3b61b45 to aee6588 Compare September 15, 2016 21:57
@weshaggard
Copy link
Copy Markdown
Member

I don't see any test failures related to the string class. I do see other errors like ApiCompat issues and such. Why do you think the string change is causing this issue?

Are we also going to change from beta-devapi to beta?

@joperezr
Copy link
Copy Markdown
Member

@weshaggard I removed my change from this PR which uses the right (beta as opposed to beta-devapi) versions and that is also where the System.Runtime.Extensions were failing. The reason I removed my change is so that dotnet-bot can update this PR once a new version of coreclr is available, since if my commit is there, it won't try to update the PR. Once my coreclr PR is merged and a new package is produced, dotnet-bot will update this PR and I'll re-add my change back to use the right versions and CI should be green.

@joperezr
Copy link
Copy Markdown
Member

@weshaggard this is one of the builds with my change and you can see how the only thing that failed there was System.Runtime.Extensions tests with the cannot find method exception for System.String.Split(Char)

@dotnet-bot
Copy link
Copy Markdown
Author

Couldn't update this pull request: Head commit committer 'Jose Perez Rodriguez' is not 'dotnet-bot'
Would have applied 'Update CoreClr, CoreFx, External, ProjectNTfs to beta-24516-01, beta-devapi-24515-01, beta-24514-00, beta-24515-00, respectively'

@joperezr
Copy link
Copy Markdown
Member

is so that dotnet-bot can update this PR once a new version of coreclr is available

apparently it didn't work even with removing my change. I'll update this PR once coreclr is available.

@dotnet-bot
Copy link
Copy Markdown
Author

Couldn't update this pull request: Head commit committer 'Jose Perez Rodriguez' is not 'dotnet-bot'
Would have applied 'Update CoreClr, CoreFx, External, ProjectNTfs to beta-24516-01, beta-devapi-24515-01, beta-24514-00, beta-24515-00, respectively'

@joperezr
Copy link
Copy Markdown
Member

Looks like coreclr version beta-24516-01 still doesn't have my changes. Will wait for next one.

@stephentoub
Copy link
Copy Markdown
Member

Will wait for next one.

Will close the PR in the meantime.

@karelz karelz modified the milestone: 1.2.0 Dec 3, 2016
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.

7 participants