-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pin System.Formats.Asn #10396
Pin System.Formats.Asn #10396
Conversation
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 think you'll also need to add one of these sections
msbuild/eng/Version.Details.xml
Lines 34 to 39 in 85d1d66
<!-- Necessary for source-build. This allows the package to be retrieved from previously-source-built artifacts | |
and flow in as dependencies of the packages produced by msbuild. --> | |
<Dependency Name="System.Reflection.MetadataLoadContext" Version="8.0.0"> | |
<Uri>https://dev.azure.com/dnceng/internal/_git/dotnet-runtime</Uri> | |
<Sha>5535e31a712343a63f5d7d796cd874e563e5ac14</Sha> | |
</Dependency> |
Co-authored-by: Rainer Sigwald <raines@microsoft.com>
I added the dependency via:
I'm bit blurry on following - and would appreciate info or documentation pointers (tagging @dotnet/source-build-internal to chime in here):
|
Since you've defined the dependency using darc, it's set up perfectly. And since this dependency comes from runtime, which is built before msbuild in the VMR, this prebuilt will only occur in the msbuild repo's build, not the VMR's build. For that reason, it's acceptable to simply configure this as an allowed prebuilt rather than defining this package in SBRP. |
Co-authored-by: Rainer Sigwald <raines@microsoft.com>
Fixes https://devdiv.visualstudio.com/DevDiv/_componentGovernance/DotNet-msbuild-Trusted/alert/9977351?typeId=6797870&pipelinesTrackingFilter=0
Context
Lets pin System.Formats.Asn, till the System.Security.Cryptography.Pkcs is available and updated