-
Notifications
You must be signed in to change notification settings - Fork 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
Only pack the shipping/non-shipping nupkgs for roslyn #67679
Conversation
Avoid packing release, pre-release stable nupkgs, and symbol nupkgs
FYI @Forgind. Note this also makes roslyn's intermediate nupkg a 3rd of its former size. |
<ItemGroup> | ||
<IntermediateNupkgArtifactFile Include="$(CurrentRepoSourceBuildArtifactsPackagesDir)Shipping\**\*.nupkg" /> | ||
<IntermediateNupkgArtifactFile Include="$(CurrentRepoSourceBuildArtifactsPackagesDir)NonShipping\**\*.nupkg" /> | ||
<!-- Don't pack any symbol packages: not needed for downstream source-build CI. |
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.
How does this stop us from packing?
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.
c6b058c#diff-6a28d92de8923be8821f2123541dc23c8ad0359c96117709e6d49e752c0303d2R9
This prevents the defaults (whole packages dir) from being included.
@jaredpar Any further concerns here? |
@mmitche - I'm seeing a similar issue with F# packages. |
ping - any update on this PR? Would be nice to get this merged in for source-build. TIA |
/azp run |
Azure Pipelines successfully started running 4 pipeline(s). |
Avoid packing release, pre-release stable nupkgs, and symbol nupkg.
Without this change, source build was preferencing the stable, release nupkgs when flowing between the repos. This worked until a new property showed up in the SDK bundled version props which is the net compiler toolset version. This should always be the non-stable version, based on the fact that this is the version used in the Microsoft build.