-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
bpo-46267: Correctly use compresslevel in gzip.compress #30416
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
Conversation
No news item is required. This new method for gzip.compress was introduced in 3.11. |
This does require a news entry if the minor version is different vs. where bug was introduced. This is true for all bugfixes. |
This bug was introduced in 3.11. I am hoping this will still get into main before python 3.11 releases. So no news item is required. |
eda7816
to
c35652c
Compare
3.11 is a major release, what was the minor release number? |
Ah, I see I was using semantic versioning terminology, but obviously that does not apply here. How shall I describe the news entry? It is a bit silly that the 3.11 final released version will have two entries describing the introduction of a performance improvement and a bugfix in it. So how would I word this? EDIT. Technically both commits will be in 3.11.0. So is a news entry still necessary? |
A news entry was added. |
ab56629
to
0a5955a
Compare
Misc/NEWS.d/next/Library/2022-01-18-08-03-19.bpo-46267.vbmuum.rst
Outdated
Show resolved
Hide resolved
Thanks for adding it, I think we are almost there. |
LGTM! |
28522e9
to
463b799
Compare
I squashed the commits. 4 commits for 12 lines was a bit too much. The code content is still the same. |
#31215 already fixed this, but didn't add tests. I'll merge this PR too to make sure we have tests. |
This is now a test-only change, so no need for a NEWS entry.
No, this PR already fixed it. Five months ago, four months before that other PR. It has had the bugfix label for 5 months. It is a bit unfortunate that not merging it sooner eventually lead to duplicated effort. But I understand keeping up is hard with the limited amount of people available. Anyway, I am glad it is in now. I also happened to be the one who introduced the bug (when I made the compress function have quite a lot less overhead). Imagine the shame of having broken something in CPython. Thanks! |
I'm sorry for the imprecise wording there. There is a lot of open PRs and it's difficult to keep up. |
Thanks for your contribution, and sorry again this PR had to wait for so long! |
Sorry for my sour initial response. As you can see I had to jump trough quite a lot of hoops to satisfy alukov for this simple one liner. Not that I mind, the PR became much better for it. But for this PR to close after 5 months due to someone else getting a oneliner fix in that did not get the same treatment... Well, it is quite an anti-climax ;-). Thanks again for merging and I hope you have a nice day! |
This is a small one-liner bug fix. I am the one responsible for introducing the bug in the first place, so I am happy I found it before 3.11 shipped.
https://bugs.python.org/issue46267
#90425