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.

Sign up for GitHub

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

Jump to bottom

bpo-26253: Add compressionlevel to tarfile stream #2962

Merged
merged 5 commits into from
Jun 25, 2022
Merged

bpo-26253: Add compressionlevel to tarfile stream #2962

merged 5 commits into from
Jun 25, 2022

Commits on May 8, 2022

  1. bpo-26253: Add compressionlevel to tarfile stream 

    `tarfile` already accepts a compressionlevel argument for creating
files. This patch adds the same for stream-based tarfile usage.
The default is 9, the value that was previously hard-coded.
    @jarondl
    jarondl committed May 8, 2022
    Configuration menu
    Copy the full SHA
    89fa205 View commit details
    Browse the repository at this point in the history

  2. bpo-26253: don't add compresslevel as an attribute 

    In tarfile, compresslevel will be simply passed around.

Also added 'versionadded' to stream compression level setting
    @jarondl
    jarondl committed May 8, 2022
    Configuration menu
    Copy the full SHA
    31b9835 View commit details
    Browse the repository at this point in the history

  3. bpo-26253: tarfile: add compresslevel unit tests 

    Check outcoming file headers when writing a compressed file.
    @jarondl
    jarondl committed May 8, 2022
    Configuration menu
    Copy the full SHA
    c217360 View commit details
    Browse the repository at this point in the history

Commits on Jun 17, 2022

  1. Apply suggestions from code review 

    Co-authored-by: Adam Turner <[email protected]>
    @serhiy-storchaka @AA-Turner
    serhiy-storchaka and AA-Turner authored Jun 17, 2022
    Configuration menu
    Copy the full SHA
    f48acbd View commit details
    Browse the repository at this point in the history

  2. Update Misc/NEWS.d/next/Library/2017-07-31-13-35-28.bpo-26253.8v_sCs.rst 

    Co-authored-by: Adam Turner <[email protected]>
    @serhiy-storchaka @AA-Turner
    serhiy-storchaka and AA-Turner authored Jun 17, 2022
    Configuration menu
    Copy the full SHA
    2b1b417 View commit details
    Browse the repository at this point in the history