Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Add System.Security.Permissions netfx facade and mark some APIs as obsolet to match desktop #26758

Merged
merged 2 commits into from
Feb 5, 2018

Conversation

safern
Copy link
Member

@safern safern commented Feb 1, 2018

@safern
Copy link
Member Author

safern commented Feb 2, 2018

@dotnet-bot test OSX x64 Debug Build please

@@ -4,16 +4,24 @@
<PropertyGroup>
<ProjectGuid>{07CAF142-B259-418E-86EF-C4BD8B50253E}</ProjectGuid>
<!-- UAPvNext is not yet mapped to netstandard2.0, manually duplicate this ref -->
<PackageTargetFramework>netstandard2.0;$(UAPvNextTFM)</PackageTargetFramework>
<PackageTargetFramework Condition="'$(TargetGroup)' == 'netstandard'">netstandard2.0;$(UAPvNextTFM)</PackageTargetFramework>
Copy link
Member

Choose a reason for hiding this comment

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

We shouldn't even need this any longer as UAP supports netstandard2.0. IIRC we removed most of these from the repo already.

<ProjectReference Include="..\..\System.Security.AccessControl\ref\System.Security.AccessControl.csproj" />
<SuppressPackageTargetFrameworkCompatibility Include="$(UAPvNextTFM)" />
Copy link
Member

Choose a reason for hiding this comment

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

This can be removed if you remove the PackageTargetFramework above.

Copy link
Member

@weshaggard weshaggard left a comment

Choose a reason for hiding this comment

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

Looks good other then the one comment.

@safern safern merged commit 2ef483f into dotnet:master Feb 5, 2018
@safern safern deleted the SystemSecurityPermissions branch February 5, 2018 22:08
@karelz karelz added this to the 2.1.0 milestone Mar 10, 2018
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corefx that referenced this pull request Sep 23, 2019
* Remove IsFastSort optimizations from String
- Fix bugs in the Span-based globalization fast paths. Note that some of these bug fixes are going to affect performance of the globalization fast paths.
- Use Span-based globalization fast paths for strings
- Avoid static array allocation for HighCharTable

Fixes dotnet#26758

Signed-off-by: dotnet-bot <[email protected]>
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corefx that referenced this pull request Sep 24, 2019
* Remove IsFastSort optimizations from String
- Fix bugs in the Span-based globalization fast paths. Note that some of these bug fixes are going to affect performance of the globalization fast paths.
- Use Span-based globalization fast paths for strings
- Avoid static array allocation for HighCharTable

Fixes dotnet#26758

Signed-off-by: dotnet-bot <[email protected]>
stephentoub pushed a commit that referenced this pull request Sep 25, 2019
* Remove IsFastSort optimizations from String
- Fix bugs in the Span-based globalization fast paths. Note that some of these bug fixes are going to affect performance of the globalization fast paths.
- Use Span-based globalization fast paths for strings
- Avoid static array allocation for HighCharTable

Fixes #26758

Signed-off-by: dotnet-bot <[email protected]>
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
…solet to match desktop (dotnet/corefx#26758)

* Add System.Security.Permissions netfx facade and mark some APIs as obsolet to match desktop

* PR Feedback


Commit migrated from dotnet/corefx@2ef483f
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.

4 participants