bpo-26978: Implement pathlib.Path.link_to (Using os.link) #12990

nanjekyejoannah · 2019-04-28T01:49:38Z

I have implemented pathlib.Path.link_to.

https://bugs.python.org/issue26978


        Implement pathlib.Path.link


        Fix documentation


        📜🤖 Added by blurb_it.

nanjekyejoannah · 2019-04-28T19:37:30Z

cc @pitrou

pitrou

Thanks for submitting this PR. You'll find some comments below.

Doc/library/pathlib.rst

Lib/pathlib.py

Lib/test/test_pathlib.py

bedevere-bot · 2019-04-29T17:47:19Z

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

And if you don't make the requested changes, you will be poked with soft cushions!


        Implement pathlib.Path.link


        Merge branch 'issue26978' of https://github.com/nanjekyejoannah/cpython…

… into issue26978

nanjekyejoannah · 2019-05-04T13:27:00Z

I have made the requested changes; please review again.

bedevere-bot · 2019-05-04T13:27:03Z

Thanks for making the requested changes!

@pitrou: please review the changes made to this pull request.

Doc/whatsnew/3.8.rst


        fix Doc


        update news entry

pitrou · 2019-05-04T15:02:58Z

Thank you @nanjekyejoannah. I will wait for CI to pass and then merge if green.

ZackerySpytz · 2019-05-04T15:22:02Z

Doc/whatsnew/3.8.rst

@@ -334,6 +334,10 @@ pathlib
 contain characters unrepresentable at the OS level.
 (Contributed by Serhiy Storchaka in :issue:`33721`.)

+Added :meth:`pathlib.Path.link_to()` which creates a hard link pointing
+to a path.
+(Contributed by Joannah Nanjekye in :issue:`26978`)


Suggested change

(Contributed by Joannah Nanjekye in :issue:`26978`)

(Contributed by Joannah Nanjekye in :issue:`26978`.)

ZackerySpytz · 2019-05-04T15:35:36Z

Misc/NEWS.d/next/Library/2019-04-28-01-52-39.bpo-26978.Lpm-SI.rst

@@ -0,0 +1,2 @@
+`pathlib.path.link_to()` is now implemented. It creates a hard link pointing


It should be pathlib.Path.link_to(), not pathlib.path.link_to().

Also, double backqoutes should be used here instead of single backquotes (as per https://devguide.python.org/documenting/#inline-markup). The following could also be used:

:meth:`pathlib.Path.link_to()`


        bpo-38811: Check for presence of os.link method in pathlib

Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (pythonGH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>


        bpo-38811: Check for presence of os.link method in pathlib

Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (pythonGH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>


        bpo-38811: Check for presence of os.link method in pathlib

Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (pythonGH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>


        bpo-38811: Check for presence of os.link method in pathlib

Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (pythonGH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>


        bpo-38811: Check for presence of os.link method in pathlib

Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (pythonGH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>


        bpo-38811: Check for presence of os.link method in pathlib

Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (pythonGH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>


        bpo-38811: Check for presence of os.link method in pathlib

Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (pythonGH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>


        bpo-38811: Check for presence of os.link method in pathlib (GH-17225)

Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (GH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com>


        bpo-38811: Check for presence of os.link method in pathlib (pythonGH-…

…17225) Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (pythonGH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com> (cherry picked from commit 092435e) Co-authored-by: Toke Høiland-Jørgensen <toke@redhat.com>


        bpo-38811: Check for presence of os.link method in pathlib (GH-17225)

Commit 6b5b013 ("bpo-26978: Implement pathlib.Path.link_to (Using os.link) (GH-12990)") introduced a new link_to method in pathlib. However, this makes pathlib crash when the 'os' module is missing a 'link' method. Fix this by checking for the presence of the 'link' method on pathlib module import, and if it's not present, turn it into a runtime error like those emitted when there is no lchmod() or symlink(). Signed-off-by: Toke Høiland-Jørgensen <toke@redhat.com> (cherry picked from commit 092435e) Co-authored-by: Toke Høiland-Jørgensen <toke@redhat.com>

nanjekyejoannah added 2 commits Apr 28, 2019

Implement pathlib.Path.link

4335641

Fix documentation

Loading status checks…

95aadda

the-knights-who-say-ni added the CLA signed label Apr 28, 2019

bedevere-bot added the awaiting review label Apr 28, 2019

📜🤖 Added by blurb_it.

Loading status checks…

51f9654

auvipy approved these changes Apr 28, 2019

View changes

bedevere-bot added awaiting core review and removed awaiting review labels Apr 28, 2019

pitrou requested changes Apr 29, 2019

View changes

Doc/library/pathlib.rst Outdated Show resolved Hide resolved

Doc/library/pathlib.rst Show resolved Hide resolved

Lib/pathlib.py Outdated Show resolved Hide resolved

Lib/test/test_pathlib.py Show resolved Hide resolved

bedevere-bot removed the awaiting core review label Apr 29, 2019

bedevere-bot added the awaiting changes label Apr 29, 2019

nanjekyejoannah added 2 commits May 4, 2019

Implement pathlib.Path.link

a418a80

Merge branch 'issue26978' of https://github.com/nanjekyejoannah/cpython…

Loading status checks…

126c89b

… into issue26978

bedevere-bot removed the awaiting changes label May 4, 2019

bedevere-bot added the awaiting change review label May 4, 2019

tirkarthi reviewed May 4, 2019

View changes

Doc/whatsnew/3.8.rst Outdated Show resolved Hide resolved

nanjekyejoannah added 2 commits May 4, 2019

fix Doc

Loading status checks…

0852e47

update news entry

Verified

This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.

GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits

Loading status checks…

3291e1c

pitrou changed the title ~~bpo-26978: Implement pathlib.Path.link (Using os.link)~~ bpo-26978: Implement pathlib.Path.link_to (Using os.link) May 4, 2019

pitrou approved these changes May 4, 2019

View changes

bedevere-bot added awaiting merge and removed awaiting change review labels May 4, 2019

ZackerySpytz reviewed May 4, 2019

View changes

bedevere-bot removed the awaiting merge label May 4, 2019

nanjekyejoannah deleted the nanjekyejoannah:issue26978 branch May 4, 2019

ZackerySpytz reviewed May 4, 2019

View changes

tohojo mentioned this pull request Nov 15, 2019

bpo-38811: Check for presence of os.link method in pathlib #17170

Merged

tohojo mentioned this pull request Nov 18, 2019

bpo-38811: Check for presence of os.link method in pathlib (v2) #17225

Merged

miss-islington mentioned this pull request Dec 16, 2019

[3.8] bpo-38811: Check for presence of os.link method in pathlib (GH-17225) #17626

Merged

python/cpython

bpo-26978: Implement pathlib.Path.link_to (Using os.link) #12990

bpo-26978: Implement pathlib.Path.link_to (Using os.link) #12990

nanjekyejoannah commented Apr 28, 2019 •

edited

This comment has been minimized.

nanjekyejoannah commented Apr 28, 2019

pitrou left a comment

This comment has been minimized.

bedevere-bot commented Apr 29, 2019

This comment has been minimized.

nanjekyejoannah commented May 4, 2019

This comment has been minimized.

bedevere-bot commented May 4, 2019

This comment has been minimized.

pitrou commented May 4, 2019

This comment has been minimized.

This comment has been minimized.

Sponsor python/cpython

python/cpython

Join GitHub today

bpo-26978: Implement pathlib.Path.link_to (Using os.link) #12990

bpo-26978: Implement pathlib.Path.link_to (Using os.link) #12990

Conversation

nanjekyejoannah commented Apr 28, 2019 • edited

This comment has been minimized.

nanjekyejoannah commented Apr 28, 2019

pitrou left a comment

This comment has been minimized.

bedevere-bot commented Apr 29, 2019

This comment has been minimized.

nanjekyejoannah commented May 4, 2019

This comment has been minimized.

bedevere-bot commented May 4, 2019

This comment has been minimized.

pitrou commented May 4, 2019

This comment has been minimized.

ZackerySpytz May 4, 2019

This comment has been minimized.

ZackerySpytz May 4, 2019

nanjekyejoannah commented Apr 28, 2019 •

edited