Skip to content

Conversation

@ericstj
Copy link
Member

@ericstj ericstj commented Aug 14, 2023

Fix #90311

Still on mono runtime this was throwing from ComWrappers.TryGetComInstance on Windows. Catch the PNSE and return false.

@ghost
Copy link

ghost commented Aug 14, 2023

Tagging subscribers to this area: @dotnet/area-system-componentmodel
See info in area-owners.md if you want to be subscribed.

Issue Details

Fix #90311

Still on mono runtime this was throwing from ComWrappers.TryGetComInstance on Windows. Catch the PNSE and return false.

Author: ericstj
Assignees: -
Labels:

area-System.ComponentModel

Milestone: -

@ivanpovazan
Copy link
Member

ivanpovazan commented Aug 14, 2023

FWIW, could you please also run runtime-extra-platforms pipeline (via: /azp run runtime-extra-platforms) to verify the fix, as the regressions were observed on win-x64 legs with Mono, which are not ran by default.

@AaronRobinsonMSFT AaronRobinsonMSFT self-requested a review August 14, 2023 23:21
@ericstj
Copy link
Member Author

ericstj commented Aug 15, 2023

Deferring to @jkotas's change to ComWrappers.

@ericstj ericstj closed this Aug 15, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Sep 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PlatformNotSupportedException thrown for System.ComponentModel.TypeDescriptor.NodeFor

4 participants