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

Add global NoWarn for: 'NU5104' #9495

Merged
merged 2 commits into from
Jun 14, 2018

Conversation

johnbeisner
Copy link

@johnbeisner johnbeisner commented Jun 14, 2018

The "pack" command under 'buildCrossTargeting' for 'Microsoft.DotNet.MSBuildSdkResolver' now throws a "NU5104" warning/error because the SDK stage0 was changed to "2.1.300" [change was intended].
Impacts: Linux-x64, Linux-arm, and Linux-arm64

Related: dotnet/source-build#569

Example error:
From: https://devdiv.visualstudio.com/DefaultCollection/DevDiv/Default/_build/results?buildId=1773593
/opt/code/.dotnet_stage0/arm/sdk/2.1.300/Sdks/NuGet.Build.Tasks.Pack/buildCrossTargeting/NuGet.Build.Tasks.Pack.targets(198,5): error NU5104: A stable release of a package should not have a prerelease dependency. Either modify the version spec of dependency "NuGet.Frameworks [4.7.0-rtm.5148, )" or update the version field in the nuspec.

Successful VSO build - Linux-arm: https://devdiv.visualstudio.com/DefaultCollection/DevDiv/Default/_build/results?buildId=1776356

Successful VSO build - Linux-arm:
dropsuffix=true
https://devdiv.visualstudio.com/DefaultCollection/DevDiv/Default/_build/results?buildId=1776659
from the log:
GenerateArchives:
/bin/tar -czf /opt/code/bin/2/linux-arm/packages/dotnet-sdk-2.1.301-linux-arm.tar.gz

…MSBuildSdkResolver' now throws a "NU5104" warning/error because the SDK stage0 was changed to "2.1.300" [change was intended].
@johnbeisner johnbeisner self-assigned this Jun 14, 2018
@johnbeisner johnbeisner requested review from livarcocc, mmitche, dagood and a team June 14, 2018 20:38
@@ -28,7 +28,7 @@

<!-- Disable asset target fallback warning globally since it does not work transitively on NoWarn of individual packages -->
<!-- Since we disabled the implict fallback to net461, this will only kick in when we have an explicit fallback and we don't need to be warned about it doing what we asked it to do. -->
<NoWarn>NU1701</NoWarn>
<NoWarn>NU1701;NU5104</NoWarn>

This comment was marked as spam.

This comment was marked as spam.

@mmitche mmitche merged commit 612c9c8 into dotnet:release/2.1.3xx Jun 14, 2018
@johnbeisner johnbeisner deleted the AddBackNoWarn branch June 20, 2018 22:51
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.

3 participants