Skip to content

Add text-only packages required for 7.0#395

Merged
MichaelSimons merged 2 commits intodotnet:mainfrom
MichaelSimons:7.0-text-only-packages
Jul 5, 2022
Merged

Add text-only packages required for 7.0#395
MichaelSimons merged 2 commits intodotnet:mainfrom
MichaelSimons:7.0-text-only-packages

Conversation

@MichaelSimons
Copy link
Copy Markdown
Member

@MichaelSimons MichaelSimons commented Jul 5, 2022

This change adds the following text-only packages:

Wcwidth.Sources 0.6.0
Microsoft.Build.NoTargets 3.5.0
Microsoft.Build.Traversal 3.1.6
Microsoft.DotNet.CilStrip.Sources 7.0.0-beta.22313.1

This change removs the following text-only packages:
Wcwidth.Sources 0.5.0
Microsoft.Build.NoTargets 3.1.0
Microsoft.Build.Traversal 3.0.23

@lbussell
Copy link
Copy Markdown
Member

lbussell commented Jul 5, 2022

Are we sure that CilStrip.Sources doesn't need to be generated at tarball creation time? That version number seems like it's in development and may be updated when net7.0 releases.

Alternatively, will this prebuilt go away with this issue being resolved? It seems like the preferable approach to me, unless I'm missing something. This is from a previous investigation we did, I think. dotnet/runtime#69460

The rest of the packages seem OK to me.

@MichaelSimons
Copy link
Copy Markdown
Member Author

This is from a previous investigation we did, I think. dotnet/runtime#69460

You are correct. This should be resolved by runtime, not SBRP. I will remove this package.

@MichaelSimons MichaelSimons merged commit 6f90699 into dotnet:main Jul 5, 2022
@MichaelSimons MichaelSimons deleted the 7.0-text-only-packages branch July 5, 2022 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants