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

Add back some surface area to thinned corelib#6247

Merged
ellismg merged 2 commits into
dotnet:masterfrom
ellismg:add-back-exception-members
Jul 13, 2016
Merged

Add back some surface area to thinned corelib#6247
ellismg merged 2 commits into
dotnet:masterfrom
ellismg:add-back-exception-members

Conversation

@ellismg
Copy link
Copy Markdown

@ellismg ellismg commented Jul 13, 2016

We are working on expsosing some existing exception types from
System.Runtime in CoreFX. The implementations will facade over
System.Private.CoreLib, but we need to make sure all the members we expect
to expose from the contract are present in the implementation assembly.

Expose the missing members for ExecutionEngineException,
NotFiniteNumberException and ExternalException.

ellismg added 2 commits July 12, 2016 17:58
On recent builds of Windows/PowerShell it seems like PS wants to write a
prompt after invoking the script, which causes a problem because that puts
extra gunk into the generated source file from h2inc, which causes
downstream failures.
We are working on expsosing some existing exception types from
System.Runtime in CoreFX. The implementations will facade over
System.Private.CoreLib, but we need to make sure all the members we expect
to expose from the contract are present in the implementation assembly.

Expose the missing members for ExecutionEngineException,
NotFiniteNumberException and ExternalException.
@ellismg
Copy link
Copy Markdown
Author

ellismg commented Jul 13, 2016

/cc @stephentoub @Lixire

Steve, once this is good to go, I will merge it and #6205 and then deal with the asmmeta fallout in TFS once the changes are mirrored over.

This is needed to unblock @Lixire adding back this stuff to CoreFX (and then we need to get a new targeting pack built which is not currently happening per build but we can fix). She's been working around the issue for now by suppressing the validation failures.

@stephentoub
Copy link
Copy Markdown
Member

Thanks, Matt. LGTM.

@danmoseley
Copy link
Copy Markdown
Member

Matt #6230 also modified model.xml recently. I was assuming that corefx would see the changes tomorrow. It sounds like this isn't, in fact, happening nightly?

@stephentoub
Copy link
Copy Markdown
Member

It sounds like this isn't, in fact, happening nightly?

Have you seen those "Updating CoreClr dependencies to beta-*" PRs from dotnet-bot? Those are what update corefx to pull in the latest coreclr. You'll also notice none of them have been merged in the last two weeks. They're all failing due to https://github.com/dotnet/corefx/issues/10014. Once that's fixed, we should be able to merge them, at which point we should be pulling in the latest CoreCLR at least once a day.

@ellismg ellismg merged commit 2272373 into dotnet:master Jul 13, 2016
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…n-members

Add back some surface area to thinned corelib

Commit migrated from dotnet/coreclr@2272373
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.

4 participants