Conversation
src/DotNetBridge/DotNetBridge.csproj
Outdated
@@ -14,6 +14,8 @@ | |||
<Copyright>(c) Microsoft Corporation. All rights reserved.</Copyright> | |||
<RepositoryUrl>https://github.com/Microsoft/NimbusML</RepositoryUrl> | |||
<PackageProjectUrl>https://github.com/Microsoft/NimbusML</PackageProjectUrl> | |||
<SignAssembly>true</SignAssembly> | |||
<AssemblyOriginatorKeyFile>tmsn.snk</AssemblyOriginatorKeyFile> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this named tmsn.snk
? that seems cryptic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you also want to set
<PublicSign Condition="'$(OS)' != 'Windows_NT'">true</PublicSign>
see
https://github.com/dotnet/machinelearning/blob/master/Directory.Build.props#L105
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I copied tmsn.snk from TLC to keep the keys the same for ML.NET external friend repos. I'll rename it to something less cryptic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like the <PublicSign
line is no longer necessary, since the bug was fixed - dotnet/roslyn#8210
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This change will allow ML.NET assemblies to add DotNetBridge as InternalsVisibleTo friend.