Add 'PackageSourceMapping' entries to Nuget.config, since multiple sources are used - #545
Conversation
adrastogi
left a comment
There was a problem hiding this comment.
Thanks, Mark Schofield (@mschofie)! This looks good to me, but I've also tagged some folks from the WASDK team to take a look in case there are implications of this change I might not be aware of.
|
This is fine - we may have to revisit when agg starts building against main branch samples (and no longer references nuget.org directly) |
|
It looks like this might break BackgroundTask samples, since they depend on version 10.0.26100.2161 of "Microsoft.Windows.SDK.BuildTools" which isn't on nuget.org. I'm trying to buddy build all samples, but it's a bit tricky :/ |
|
Hmm... So "Microsoft.Windows.SDK.BuildTools" version "10.0.26100.2161" isn't available on either source:
So the problems I'm seeing with that are unrelated to my changes. |
Thanks Mark - totally up to you if you want to update those references to latest, 10.0.26100.6584 |
|
Thanks, Scott, I've filed #558 to track the work. I think that completes due diligence for this PR. |
Description
When building the WindowsML samples through the command-line with:
.\build.cmd x64 Debug WindowsMLI see errors, along the lines of:
The error is complaining because there's a
:/Samples/nuget.configfile with multiplepackageSourceelements, but nopackageSourceMapping. So this PR addspackageSourceMappingfor thepackageSourceelements.Note: The samples are still not building cleanly, there's more errors behind this one.
Checklist
Note that /azp run currently isn't working for this repo.