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

Move files to shared CoreLib partition#10837

Merged
jkotas merged 1 commit intodotnet:masterfrom
jkotas:shared
Apr 10, 2017
Merged

Move files to shared CoreLib partition#10837
jkotas merged 1 commit intodotnet:masterfrom
jkotas:shared

Conversation

@jkotas
Copy link
Member

@jkotas jkotas commented Apr 10, 2017

  • Reconcile diffs with CoreRT as appropriate
  • Deleted unused internal SuppressMergeCheckAttribute and XmlIgnoreMemberAttribute

<!-- Setting this to "false" works only for workstation GC, not server. -->
<FeatureSymDiff>true</FeatureSymDiff>
<FeatureReadyToRun Condition="'$(TargetArch)'!='arm64'">true</FeatureReadyToRun>
<FeatureReadyToRun>true</FeatureReadyToRun>
Copy link
Member Author

Choose a reason for hiding this comment

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

Mismatch between C++ makefiles and C# project files that I run into

Copy link
Member

Choose a reason for hiding this comment

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

Does this change behavior -- is it worth a separate PR in case it needs revert?

Copy link
Member Author

Choose a reason for hiding this comment

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

will do

@jkotas jkotas requested a review from danmoseley April 10, 2017 04:00
{
internal unsafe partial class Sys
{
[DllImport(Interop.Libraries.SystemNative, EntryPoint = "SystemNative_GetNonCryptographicallySecureRandomBytes")]
Copy link
Member

Choose a reason for hiding this comment

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

I can't find this entry point.. mabye it's a generated name?

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Oh it's in corefx. I don't understand how that kind of call works yet.

Copy link
Member

@danmoseley danmoseley left a comment

Choose a reason for hiding this comment

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

LGTM

@jkotas jkotas merged commit 2516992 into dotnet:master Apr 10, 2017
dotnet-bot pushed a commit to dotnet/corert that referenced this pull request Apr 10, 2017
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
jkotas added a commit to dotnet/corert that referenced this pull request Apr 10, 2017
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
@jkotas jkotas deleted the shared branch April 10, 2017 16:29
@karelz karelz modified the milestone: 2.0.0 Aug 28, 2017
dotnet-bot pushed a commit to dotnet/corefx that referenced this pull request Jan 13, 2018
Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>
dotnet-bot pushed a commit to dotnet/corefx that referenced this pull request Jan 13, 2018
Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>
safern pushed a commit to dotnet/corefx that referenced this pull request Jan 16, 2018
Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>
safern pushed a commit to dotnet/corefx that referenced this pull request Jan 16, 2018
Signed-off-by: dotnet-bot-corefx-mirror <dotnet-bot@microsoft.com>
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