Skip to content

Conversation

@MaximLipnin
Copy link
Contributor

@MaximLipnin MaximLipnin commented Aug 10, 2020

Even though the related spec PR is a draft, we could start looking at how to use that new UnsupportedOSPlatform attribute.

APIs marked as unsupported at assembly level in this PR:

  • System.Security.Cryptography.* besides S.S.C.Algorithms - it's mostly PNSE on browser except RandomNumberGenerator and RandomNumberGeneratorImplementation classes so we'll have to mark all the remaining types as unsupported
  • System.Data.Odbc
  • System.DirectoryServices.Protocols
  • System.Drawing.Common
  • System.IO.Ports

The example of the assembly info generated for S.S.C.OpenSsl which contains UnsupportedOSPlatform attributes for windows and browser platforms:
image

Part of #41087

@ghost
Copy link

ghost commented Aug 10, 2020

Tagging subscribers to this area: @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.

@MaximLipnin MaximLipnin force-pushed the wasm_add_attr_for_unsupported_api branch from 27c710e to d9fe14b Compare August 11, 2020 07:54
@MaximLipnin MaximLipnin marked this pull request as ready for review August 13, 2020 14:05
@steveisok steveisok merged commit 196fd18 into dotnet:master Aug 13, 2020
@MaximLipnin MaximLipnin deleted the wasm_add_attr_for_unsupported_api branch August 14, 2020 06:28
@jeffhandley
Copy link
Member

This is already merged, but tagging @adamsitnik as an FYI.

@jeffhandley
Copy link
Member

Very surgical; nice!

Copy link
Member

@adamsitnik adamsitnik left a comment

Choose a reason for hiding this comment

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

great work! 👍

@karelz karelz added this to the 5.0.0 milestone Aug 18, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 7, 2020
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.

7 participants