Skip to content
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

Update PackageOverrides.txt #104638

Merged
merged 2 commits into from
Jul 15, 2024
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
206 changes: 117 additions & 89 deletions src/installer/pkg/sfx/Microsoft.NETCore.App/PackageOverrides.txt
Original file line number Diff line number Diff line change
@@ -1,91 +1,97 @@
Microsoft.CSharp|4.4.0
Microsoft.CSharp|4.7.0
Microsoft.VisualBasic|10.4.0
Microsoft.Win32.Primitives|4.3.0
Microsoft.Win32.Registry|4.4.0
runtime.debian.8-x64.runtime.native.System|4.3.0
runtime.debian.8-x64.runtime.native.System.IO.Compression|4.3.0
runtime.debian.8-x64.runtime.native.System.Net.Http|4.3.0
runtime.debian.8-x64.runtime.native.System.Net.Security|4.3.0
runtime.debian.8-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.debian.8-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
runtime.fedora.23-x64.runtime.native.System|4.3.0
runtime.fedora.23-x64.runtime.native.System.IO.Compression|4.3.0
runtime.fedora.23-x64.runtime.native.System.Net.Http|4.3.0
runtime.fedora.23-x64.runtime.native.System.Net.Security|4.3.0
runtime.fedora.23-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.fedora.23-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
runtime.fedora.24-x64.runtime.native.System|4.3.0
runtime.fedora.24-x64.runtime.native.System.IO.Compression|4.3.0
runtime.fedora.24-x64.runtime.native.System.Net.Http|4.3.0
runtime.fedora.24-x64.runtime.native.System.Net.Security|4.3.0
runtime.fedora.24-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.fedora.24-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
runtime.opensuse.13.2-x64.runtime.native.System|4.3.0
runtime.opensuse.13.2-x64.runtime.native.System.IO.Compression|4.3.0
runtime.opensuse.13.2-x64.runtime.native.System.Net.Http|4.3.0
runtime.opensuse.13.2-x64.runtime.native.System.Net.Security|4.3.0
runtime.opensuse.13.2-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.opensuse.13.2-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
runtime.opensuse.42.1-x64.runtime.native.System|4.3.0
runtime.opensuse.42.1-x64.runtime.native.System.IO.Compression|4.3.0
runtime.opensuse.42.1-x64.runtime.native.System.Net.Http|4.3.0
runtime.opensuse.42.1-x64.runtime.native.System.Net.Security|4.3.0
runtime.opensuse.42.1-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.opensuse.42.1-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
runtime.osx.10.10-x64.runtime.native.System|4.3.0
runtime.osx.10.10-x64.runtime.native.System.IO.Compression|4.3.0
runtime.osx.10.10-x64.runtime.native.System.Net.Http|4.3.0
runtime.osx.10.10-x64.runtime.native.System.Net.Security|4.3.0
runtime.osx.10.10-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.osx.10.10-x64.runtime.native.System.Security.Cryptography.Apple|4.3.0
runtime.osx.10.10-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
runtime.rhel.7-x64.runtime.native.System|4.3.0
runtime.rhel.7-x64.runtime.native.System.IO.Compression|4.3.0
runtime.rhel.7-x64.runtime.native.System.Net.Http|4.3.0
runtime.rhel.7-x64.runtime.native.System.Net.Security|4.3.0
runtime.rhel.7-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.rhel.7-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
runtime.ubuntu.14.04-x64.runtime.native.System|4.3.0
runtime.ubuntu.14.04-x64.runtime.native.System.IO.Compression|4.3.0
runtime.ubuntu.14.04-x64.runtime.native.System.Net.Http|4.3.0
runtime.ubuntu.14.04-x64.runtime.native.System.Net.Security|4.3.0
runtime.ubuntu.14.04-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.ubuntu.14.04-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
runtime.ubuntu.16.04-x64.runtime.native.System|4.3.0
runtime.ubuntu.16.04-x64.runtime.native.System.IO.Compression|4.3.0
runtime.ubuntu.16.04-x64.runtime.native.System.Net.Http|4.3.0
runtime.ubuntu.16.04-x64.runtime.native.System.Net.Security|4.3.0
runtime.ubuntu.16.04-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.ubuntu.16.04-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
runtime.ubuntu.16.10-x64.runtime.native.System|4.3.0
runtime.ubuntu.16.10-x64.runtime.native.System.IO.Compression|4.3.0
runtime.ubuntu.16.10-x64.runtime.native.System.Net.Http|4.3.0
runtime.ubuntu.16.10-x64.runtime.native.System.Net.Security|4.3.0
runtime.ubuntu.16.10-x64.runtime.native.System.Security.Cryptography|4.3.0
runtime.ubuntu.16.10-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.0
Microsoft.Win32.Registry|5.0.0
runtime.debian.8-x64.runtime.native.System|4.3.1
runtime.debian.8-x64.runtime.native.System.IO.Compression|4.3.2
runtime.debian.8-x64.runtime.native.System.Net.Http|4.3.1
runtime.debian.8-x64.runtime.native.System.Net.Security|4.3.1
runtime.debian.8-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.debian.8-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
runtime.fedora.23-x64.runtime.native.System|4.3.1
runtime.fedora.23-x64.runtime.native.System.IO.Compression|4.3.2
runtime.fedora.23-x64.runtime.native.System.Net.Http|4.3.1
runtime.fedora.23-x64.runtime.native.System.Net.Security|4.3.1
runtime.fedora.23-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.fedora.23-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
runtime.fedora.24-x64.runtime.native.System|4.3.1
runtime.fedora.24-x64.runtime.native.System.IO.Compression|4.3.2
runtime.fedora.24-x64.runtime.native.System.Net.Http|4.3.1
runtime.fedora.24-x64.runtime.native.System.Net.Security|4.3.1
runtime.fedora.24-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.fedora.24-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
runtime.opensuse.13.2-x64.runtime.native.System|4.3.1
runtime.opensuse.13.2-x64.runtime.native.System.IO.Compression|4.3.2
runtime.opensuse.13.2-x64.runtime.native.System.Net.Http|4.3.1
runtime.opensuse.13.2-x64.runtime.native.System.Net.Security|4.3.1
runtime.opensuse.13.2-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.opensuse.13.2-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
runtime.opensuse.42.1-x64.runtime.native.System|4.3.1
runtime.opensuse.42.1-x64.runtime.native.System.IO.Compression|4.3.2
runtime.opensuse.42.1-x64.runtime.native.System.Net.Http|4.3.1
runtime.opensuse.42.1-x64.runtime.native.System.Net.Security|4.3.1
runtime.opensuse.42.1-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.opensuse.42.1-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
runtime.osx.10.10-x64.runtime.native.System|4.3.1
runtime.osx.10.10-x64.runtime.native.System.IO.Compression|4.3.2
runtime.osx.10.10-x64.runtime.native.System.Net.Http|4.3.1
runtime.osx.10.10-x64.runtime.native.System.Net.Security|4.3.1
runtime.osx.10.10-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.osx.10.10-x64.runtime.native.System.Security.Cryptography.Apple|4.3.1
runtime.osx.10.10-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
runtime.rhel.7-x64.runtime.native.System|4.3.1
runtime.rhel.7-x64.runtime.native.System.IO.Compression|4.3.2
runtime.rhel.7-x64.runtime.native.System.Net.Http|4.3.1
runtime.rhel.7-x64.runtime.native.System.Net.Security|4.3.1
runtime.rhel.7-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.rhel.7-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
runtime.ubuntu.14.04-x64.runtime.native.System|4.3.1
runtime.ubuntu.14.04-x64.runtime.native.System.IO.Compression|4.3.2
runtime.ubuntu.14.04-x64.runtime.native.System.Net.Http|4.3.1
runtime.ubuntu.14.04-x64.runtime.native.System.Net.Security|4.3.1
runtime.ubuntu.14.04-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.ubuntu.14.04-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
runtime.ubuntu.16.04-x64.runtime.native.System|4.3.1
runtime.ubuntu.16.04-x64.runtime.native.System.IO.Compression|4.3.2
runtime.ubuntu.16.04-x64.runtime.native.System.Net.Http|4.3.1
runtime.ubuntu.16.04-x64.runtime.native.System.Net.Security|4.3.1
runtime.ubuntu.16.04-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.ubuntu.16.04-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
runtime.ubuntu.16.10-x64.runtime.native.System|4.3.1
runtime.ubuntu.16.10-x64.runtime.native.System.IO.Compression|4.3.2
runtime.ubuntu.16.10-x64.runtime.native.System.Net.Http|4.3.1
runtime.ubuntu.16.10-x64.runtime.native.System.Net.Security|4.3.1
runtime.ubuntu.16.10-x64.runtime.native.System.Security.Cryptography|4.3.4
runtime.ubuntu.16.10-x64.runtime.native.System.Security.Cryptography.OpenSsl|4.3.3
System.AppContext|4.3.0
System.Buffers|4.4.0
System.Buffers|5.0.0
System.Collections|4.3.0
System.Collections.Concurrent|4.3.0
System.Collections.Immutable|1.4.0
System.Collections.Immutable|9.0.0
System.Collections.NonGeneric|4.3.0
System.Collections.Specialized|4.3.0
System.ComponentModel|4.3.0
System.ComponentModel.Annotations|4.3.0
System.ComponentModel.EventBasedAsync|4.3.0
System.ComponentModel.Primitives|4.3.0
System.ComponentModel.TypeConverter|4.3.0
System.Console|4.3.0
System.Console|4.3.1
System.Data.Common|4.3.0
System.Data.DataSetExtensions|4.4.0
System.Diagnostics.Contracts|4.3.0
System.Diagnostics.Debug|4.3.0
System.Diagnostics.DiagnosticSource|4.4.0
System.Diagnostics.DiagnosticSource|9.0.0
System.Diagnostics.FileVersionInfo|4.3.0
System.Diagnostics.Process|4.3.0
System.Diagnostics.StackTrace|4.3.0
System.Diagnostics.TextWriterTraceListener|4.3.0
System.Diagnostics.Tools|4.3.0
System.Diagnostics.TraceSource|4.3.0
System.Diagnostics.Tracing|4.3.0
System.Drawing.Primitives|4.3.0
System.Dynamic.Runtime|4.3.0
System.Formats.Asn1|9.0.0
System.Formats.Tar|9.0.0
System.Globalization|4.3.0
System.Globalization.Calendars|4.3.0
System.Globalization.Extensions|4.3.0
Expand All @@ -94,36 +100,52 @@ System.IO.Compression|4.3.0
System.IO.Compression.ZipFile|4.3.0
System.IO.FileSystem|4.3.0
System.IO.FileSystem.AccessControl|4.4.0
System.IO.FileSystem.DriveInfo|4.3.0
System.IO.FileSystem.DriveInfo|4.3.1
System.IO.FileSystem.Primitives|4.3.0
System.IO.FileSystem.Watcher|4.3.0
System.IO.IsolatedStorage|4.3.0
System.IO.MemoryMappedFiles|4.3.0
System.IO.Pipes|4.3.0
System.IO.Pipes.AccessControl|5.0.0
System.IO.Pipelines|9.0.0
System.IO.UnmanagedMemoryStream|4.3.0
System.Linq|4.3.0
System.Linq.Expressions|4.3.0
System.Linq.Parallel|4.3.0
System.Linq.Queryable|4.3.0
System.Net.Http|4.3.0
System.Memory|5.0.0
System.Net.Http|4.3.4
System.Net.Http.Json|9.0.0
System.Net.NameResolution|4.3.0
System.Net.Primitives|4.3.0
System.Net.NetworkInformation|4.3.0
System.Net.Ping|4.3.0
System.Net.Primitives|4.3.1
System.Net.Requests|4.3.0
System.Net.Security|4.3.0
System.Net.Security|4.3.2
System.Net.Sockets|4.3.0
System.Net.WebHeaderCollection|4.3.0
System.Net.WebSockets|4.3.0
System.Net.WebSockets.Client|4.3.2
System.Numerics.Vectors|5.0.0
System.ObjectModel|4.3.0
System.Private.DataContractSerialization|4.3.0
System.Private.Uri|4.3.2
System.Reflection|4.3.0
System.Reflection.Emit|4.3.0
System.Reflection.Emit.ILGeneration|4.3.0
System.Reflection.Emit.Lightweight|4.3.0
System.Reflection.DispatchProxy|6.0.0
System.Reflection.Emit|4.7.0
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this one can stay at 5.0.0, we never shipped a 5.0.0 (just 5.0.0-preview packages for a while), and it makes it consistent with the other 4.x packages.

System.Reflection.Emit.ILGeneration|4.7.0
System.Reflection.Emit.Lightweight|4.7.0
System.Reflection.Extensions|4.3.0
System.Reflection.Metadata|1.5.0
System.Reflection.Metadata|9.0.0
System.Reflection.Primitives|4.3.0
System.Reflection.TypeExtensions|4.3.0
System.Resources.Reader|4.3.0
System.Resources.ResourceManager|4.3.0
System.Runtime|4.3.0
System.Runtime.Extensions|4.3.0
System.Resources.Writer|4.3.0
System.Runtime|4.3.1
System.Runtime.CompilerServices.Unsafe|7.0.0
System.Runtime.CompilerServices.VisualC|4.3.0
System.Runtime.Extensions|4.3.1
System.Runtime.Handles|4.3.0
System.Runtime.InteropServices|4.3.0
System.Runtime.InteropServices.RuntimeInformation|4.3.0
Expand All @@ -132,33 +154,39 @@ System.Runtime.Numerics|4.3.0
System.Runtime.Serialization.Formatters|4.3.0
System.Runtime.Serialization.Json|4.3.0
System.Runtime.Serialization.Primitives|4.3.0
System.Security.AccessControl|4.4.0
System.Runtime.Serialization.Xml|4.3.0
System.Security.AccessControl|6.0.1
System.Security.Claims|4.3.0
System.Security.Cryptography.Algorithms|4.3.0
System.Security.Cryptography.Cng|4.4.0
System.Security.Cryptography.Algorithms|4.3.1
System.Security.Cryptography.Cng|5.0.0
System.Security.Cryptography.Csp|4.3.0
System.Security.Cryptography.Encoding|4.3.0
System.Security.Cryptography.OpenSsl|4.4.0
System.Security.Cryptography.OpenSsl|5.0.0
System.Security.Cryptography.Primitives|4.3.0
System.Security.Cryptography.X509Certificates|4.3.0
System.Security.Cryptography.Xml|4.4.0
System.Security.Cryptography.X509Certificates|4.3.2
Copy link
Member Author

@ViktorHofer ViktorHofer Jul 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From what I can see, this line is wrong as System.Security.Cryptography.Xml was never inbox.

Copy link
Member

@eerhardt eerhardt Jul 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that it is "inbox" in Microsoft.AspNetCore.App (which means its not inbox in Microsoft.NETCore.App)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's in ASP.NET Core but not .NETCore. Surprising that no-one ever hit this, maybe because the package version was old? Or folks listen to us and don't actually use it. I confirmed your analysis in 2.0 - 9.0. Thanks for catching it and removing it.

Copy link
Member Author

@ViktorHofer ViktorHofer Jul 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eerhardt I see that an entry for it is missing in https://github.com/dotnet/aspnetcore/blob/main/eng/PackageOverrides.txt. There might be others that are missing. Should I file an issue?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like ASP.NET's should be updated to no longer include System.IO.Pipelines, since that is now in NETCore.App.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@eerhardt will you take care of updating the PackageOverrides list in aspnetcore or do you want me to file an issue?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've opened dotnet/aspnetcore#56724 to track updating aspnetcore's file.

System.Security.Principal|4.3.0
System.Security.Principal.Windows|4.4.0
System.Security.Principal.Windows|5.0.0
System.Security.SecureString|4.3.0
System.Text.Encoding|4.3.0
System.Text.Encoding.CodePages|9.0.0
System.Text.Encoding.Extensions|4.3.0
System.Text.RegularExpressions|4.3.0
System.Text.Encodings.Web|9.0.0
System.Text.Json|9.0.0
System.Text.RegularExpressions|4.3.1
Comment on lines +173 to +174
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These 9.0.0 ones are interesting. Is there a reason they weren't included to begin with?

Copy link
Member Author

@ViktorHofer ViktorHofer Jul 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good question. System.Text.Json is a new package but System.Text.Encodings.Web existed since 2015 and has been inbox since at least netcoreapp2.0. If I'm not mistaken it was you who created the initial list?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This list hasn't changed since you originally added it. I think we were letting platform manifest do the work then.

With PackageOverrides we can short circuit that work. It can also help if we eventually feed this data into NuGet for supplied by platform (short circuiting even earlier in restore).

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I'm not mistaken it was you who created the initial list?

I believe it was initially added in dotnet/core-setup#7369.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

System.Threading|4.3.0
System.Threading.Channels|9.0.0
System.Threading.Overlapped|4.3.0
System.Threading.Tasks|4.3.0
System.Threading.Tasks.Extensions|4.3.0
System.Threading.Tasks.Dataflow|9.0.0
System.Threading.Tasks.Extensions|5.0.0
System.Threading.Tasks.Parallel|4.3.0
System.Threading.Thread|4.3.0
System.Threading.ThreadPool|4.3.0
System.Threading.Timer|4.3.0
System.ValueTuple|4.3.0
System.Xml.ReaderWriter|4.3.0
System.ValueTuple|4.5.0
System.Xml.ReaderWriter|4.3.1
System.Xml.XDocument|4.3.0
System.Xml.XmlDocument|4.3.0
System.Xml.XmlSerializer|4.3.0
System.Xml.XPath|4.3.0
System.Xml.XPath.XDocument|4.3.0
System.Xml.XPath.XDocument|5.0.0
Loading