Skip to content

Conversation

@mirespace
Copy link
Contributor

Hi!

This PR comes from a patch I'm using on v6.0 (it arose here #68212 ). In the thread, I was encouraged to make the PR against 7.0+, but I can make it also for v6.0 if anyone finds it useful.

Thanks in advance for considering this.

…ression.Native.so contains an unresolvable reference to symbol log2: it's probably a plugin
@ghost ghost added area-System.IO.Compression community-contribution Indicates that the PR has been added by a community member labels Jul 25, 2022
@ghost
Copy link

ghost commented Jul 25, 2022

Tagging subscribers to this area: @dotnet/area-system-io-compression
See info in area-owners.md if you want to be subscribed.

Issue Details

Hi!

This PR comes from a patch I'm using on v6.0 (it arose here #68212 ). In the thread, I was encouraged to make the PR against 7.0+, but I can make it also for v6.0 if anyone finds it useful.

Thanks in advance for considering this.

Author: mirespace
Assignees: -
Labels:

area-System.IO.Compression

Milestone: -

Copy link
Member

@am11 am11 left a comment

Choose a reason for hiding this comment

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

LGTM, thank you!

Since this is a non-fatal warning and not causing errors (e.g. due to non-zero exit status) in Ubuntu builds, it might not meet the 6.0 backport bar.

@jkotas jkotas merged commit f175438 into dotnet:main Jul 25, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Aug 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.IO.Compression community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants