Repository navigation
gh-156568: Make zipapp archives executable with PathLike targets - #156569
Conversation
|
@pfmoore fyi |
|
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 |
…kxLY8.rst Co-authored-by: Paul Moore <p.f.moore@gmail.com>
Co-authored-by: Paul Moore <p.f.moore@gmail.com>
|
I have made the requested changes; please review again |
|
Thanks for making the requested changes! @pfmoore: please review the changes made to this pull request. |
|
@pfmoore Just following up on this PR — would appreciate a review when you get a chance. Thanks! |
|
@hugovk Should I hold off on merging this as we're so close to 3.15 release (or at least hold off on the 3.15 backport)? It's not an urgent fix, so I'm personally happy to wait till 3.15 is released. |
|
You can merge it to |
|
Given that we're only a couple of weeks from 3.15 final, I'll wait - otherwise I'll probably just forget to do the 3.15 backport 🙁 |
|
Thanks @eoan-ermine for the PR, and @pfmoore for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14, 3.15. |
|
GH-159404 is a backport of this pull request to the 3.15 branch. |
|
GH-159405 is a backport of this pull request to the 3.14 branch. |
|
GH-159406 is a backport of this pull request to the 3.13 branch. |
|
Thanks for the contribution, @eoan-ermine! |
|
Backports merged, with the exception of 3.13 as that is now in security-only mode. |
|
Fix
zipapp.create_archive()not setting the executable bit for PathLike targets.zipapp.create_archive#156568