Skip to content

Fix spacing in tarfile docs #20601

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

Closed
wants to merge 1,476 commits into from
Closed

Fix spacing in tarfile docs #20601

wants to merge 1,476 commits into from

Conversation

HarshaLaxman
Copy link
Contributor

Before

content.txt is 42 bytes in size and isa regular file.
folder is 420 bytes in size and isa directory.
magic is 4200 bytes in size and issomething else.

After:

content.txt is 42 bytes in size and is a regular file.
folder is 420 bytes in size and is a directory.
magic is 4200 bytes in size and is something else.

miss-islington and others added 30 commits February 24, 2020 07:06
GH- [bpo-39128](https://bugs.python.org/issue39128): happy eyeballs description

GH- [3.9] 39128 - happy eyeballs description (GH-18624)
GH- [3.8] 39128 - happy eyeballs description (GH-18624)

https://bugs.python.org/issue39128
(cherry picked from commit 8af4712)

Co-authored-by: idomic <[email protected]>
Trying to decode an invalid string with the punycode codec
shoud raise UnicodeError.

(cherry picked from commit ba22e8f)

Co-authored-by: Berker Peksag <[email protected]>
(cherry picked from commit 57c7a0b)

Co-authored-by: Inada Naoki <[email protected]>
Previously, the button-up part of selecting with a mouse was treated as a click
that meant 'jump' to this line, which modified the context and undid the selection
(cherry picked from commit c705fd1)

Co-authored-by: Terry Jan Reedy <[email protected]>
Also fixes some potential Nuget build issues.
(cherry picked from commit 03153dd)

Co-authored-by: Steve Dower <[email protected]>
…H-18338)

* bpo-39548: Fix handling of 'WWW-Authenticate' header for Digest authentication

 - The 'qop' value in the 'WWW-Authenticate' header is optional. The
   presence of 'qop' in the header should be checked before its value
   is parsed with 'split'.

Signed-off-by: Stephen Balousek <[email protected]>

* bpo-39548: Fix handling of 'WWW-Authenticate' header for Digest authentication

 - Add NEWS item

Signed-off-by: Stephen Balousek <[email protected]>

* Update Misc/NEWS.d/next/Library/2020-02-06-05-33-52.bpo-39548.DF4FFe.rst

Co-Authored-By: Brandt Bucher <[email protected]>

Co-authored-by: Brandt Bucher <[email protected]>
(cherry picked from commit 5e260e0)

Co-authored-by: Stephen Balousek <[email protected]>
Fix compileall.compile_dir() ddir= behavior on sub-packages.

Fixes compileall.compile_dir's ddir parameter and compileall command
line flag `-d` to no longer write the wrong pathname to the generated
pyc file for submodules beneath the root of the directory tree being
compiled.  This fixes a regression introduced with Python 3.5.

Tests backported from GH 0267335, the
implementation is different due to intervening code changes.  But still
quiet simple.

Why was the bug ever introduced?  The refactoring to add parallel
execution kept the ddir -> dfile computations but discarded the results
instead of sending them to compile_file().  This fixes that.  Lack of tests
meant this went unnoticed.
Automerge-Triggered-By: @pablogsal.
(cherry picked from commit 217dce9)

Co-authored-by: Hakan Çelik <[email protected]>
…ered with exception raised. (GH-18656). (GH-18732)

(cherry picked from commit 28d0bca)
_PyEval_SetAsyncGenFinalizer() and _PyEval_SetAsyncGenFirstiter()
didn't include proper error handling for their PySys_Audit() calls.

Co-authored-by: Zackery Spytz <[email protected]>
…-17666)

Open issue in the BPO indicated a desire to make the implementation of
codecs.open() at parity with io.open(), which implements a try/except to
assure file stream gets closed before an exception is raised.
(cherry picked from commit 2565ede)

Co-authored-by: Chris A <[email protected]>
Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
(cherry picked from commit 4482337)

Co-authored-by: Lidi Zheng <[email protected]>
Automerge-Triggered-By: @zooba
(cherry picked from commit 1382c32)

Co-authored-by: Erlend Egeberg Aasland <[email protected]>
…_traverse and tp_clear (GH-18749) (GH-18756)

Objects do not own weak references to them directly through the __weakref__ list so these
do not need to be traversed by the GC.

(cherry picked from commit 0c2b509)
…dows (GH-18724)

(cherry picked from commit ce3a498)

Co-authored-by: Steve Dower <[email protected]>
…H-18761)

(cherry picked from commit 2d2f855)

Co-authored-by: Serhiy Storchaka <[email protected]>
Automerge-Triggered-By: @pablogsal
(cherry picked from commit 6df421f)

Co-authored-by: Pablo Galindo <[email protected]>
…#18752)

- Threads created by PyGILState_Ensure() could have a duplicate tstate->id.

(cherry picked from commit b3b9ade)
akuchling and others added 9 commits May 29, 2020 04:59
…) (GH-20092)

Backporting to 3.8 and adding a NEWS item (I should have added one to the master branch -- oh well).
(cherry picked from commit c610d97)

Co-authored-by: Erlend Egeberg Aasland <[email protected]>
Leave selection when right click within.  This exception to clearing selections when right-clicking was omitted from the previous commit, 4ca060d.  I did not realize that this completely disabled the context menu entries, and  I should have merged a minimal fix immediately.  An automated test should follow.
(cherry picked from commit 97e4e0f)

Co-authored-by: Terry Jan Reedy <[email protected]>
…GH-20149)

Reference to PySide has been removed has it is for Qt 4, which has reached end of life.
(cherry picked from commit 4649202)

Co-authored-by: Samuel Gaist <[email protected]>
Fix :mod:`ssl`` code to be compatible with OpenSSL 1.1.x builds that use
``no-deprecated`` and ``--api=1.1.0``.

Note: Tests assume full OpenSSL API and fail with limited API.

Signed-off-by: Christian Heimes <[email protected]>
Co-authored-by: Mark Wright <[email protected]>
(cherry picked from commit a871f69)

Co-authored-by: Christian Heimes <[email protected]>
Before
```
content.txt is 42 bytes in size and isa regular file.
folder is 420 bytes in size and isa directory.
magic is 4200 bytes in size and issomething else.
```

After:
```
content.txt is 42 bytes in size and is a regular file.
folder is 420 bytes in size and is a directory.
magic is 4200 bytes in size and is something else.
```
@the-knights-who-say-ni
Copy link

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@HarshaLaxman

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

Copy link
Contributor

@remilapeyre remilapeyre left a comment

Choose a reason for hiding this comment

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

Hi @HarshaLaxman, thanks for contributing to Python!

Like @the-knights-who-say-ni said, you will need to sign the CLA so that this contribution can be accepted in Python. You can follow the steps given in the Python Dev Guide.

You will also need to change the target branch of your Pull Request, all PR are made against master and your fix will be backported to the maintenance branches once it's accepted.

@@ -784,7 +784,7 @@ How to read a gzip compressed tar archive and display some member information::
import tarfile
tar = tarfile.open("sample.tar.gz", "r:gz")
for tarinfo in tar:
print(tarinfo.name, "is", tarinfo.size, "bytes in size and is", end="")
print(tarinfo.name, "is", tarinfo.size, "bytes in size and is ", end="")
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe this statement could use an f-string?

Copy link
Member

Choose a reason for hiding this comment

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

Ack. Still this functional improvement is valuable.

Copy link
Member

@orsenthil orsenthil left a comment

Choose a reason for hiding this comment

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

LGTM.

@orsenthil
Copy link
Member

@HarshaLaxman , please sign the CLA

@HarshaLaxman HarshaLaxman changed the base branch from 3.8 to master June 3, 2020 15:30
@HarshaLaxman
Copy link
Contributor Author

I changed the target branch to Master as requested.

I did sign the CLA yesterday with bugs.python.org username harshalaxman and GitHub name HarshaLaxman... maybe it takes some time to register?

@remilapeyre
Copy link
Contributor

remilapeyre commented Jun 3, 2020

I changed the target branch to Master as requested.

I did sign the CLA yesterday with bugs.python.org username harshalaxman and GitHub name HarshaLaxman... maybe it takes some time to register?

Please make sure your GitHub username is listed in the “Your Details” section at b.p.o: https://cloud.githubusercontent.com/assets/2680980/23276970/d14a380c-f9d1-11e6-883d-e13b6b211239.png

Also, thanks for changing the target branch but there should be only one commit in the PR, it took all changes that were in the branch.

@HarshaLaxman
Copy link
Contributor Author

I changed the target branch to Master as requested.
I did sign the CLA yesterday with bugs.python.org username harshalaxman and GitHub name HarshaLaxman... maybe it takes some time to register?

Please make sure your GitHub username is listed in the “Your Details” section at b.p.o: https://cloud.githubusercontent.com/assets/2680980/23276970/d14a380c-f9d1-11e6-883d-e13b6b211239.png

Also, thanks for changing the target branch but there should be only one commit in the PR, it took all changes that were in the branch.

My GitHub username is in the "Your Details" section, but even b.p.o doesn't recognize that I signed the CLA. Comparing to the screenshot you shared, my page does not include "Contributor Form Received", although I got email confirmation and ewa was cc'd.

For changing the target branch, I can create a new PR against master once I figure out this CLA stuff?

@orsenthil
Copy link
Member

For changing the target branch, I can create a new PR against master

Something is wrong with this PR. I suggest, pushing (force pushing) a clean version, or dropping this to create a new PR. Only one PR against master branch is enough.

@HarshaLaxman
Copy link
Contributor Author

HarshaLaxman commented Jun 4, 2020

Looks like my CLA is acknowledged now.
Created a PR against master here: #20629

This can be closed, thanks for the assistance :)

@orsenthil orsenthil closed this Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting merge docs Documentation in the Doc dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.