-
-
Notifications
You must be signed in to change notification settings - Fork 31.6k
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
bpo-28494: Test existing zipfile working behavior. #15853
Conversation
Adds unittests for executables with a zipfile appended to test_zipfile as zipfile.is_zipfile and zipfile.ZipFile work properly on these today.
duh, fixing the test to skip the execute one on windows :) |
@@ -2496,11 +2496,15 @@ def test_read_zip64_with_exe_prepended(self): | |||
self._test_zip_works(self.exe_zip64) | |||
|
|||
@unittest.skipUnless(sys.executable, 'sys.executable required.') | |||
@unittest.skipUnless(os.access('/bin/bash', os.X_OK), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How about using shutil.which('bash') instead? There are some windows machines with bash installed now... will the test work there?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't think windows supported #! line executables as a concept. executable is based on filename ending in bat/cmd/com/exe, not an execute bit there.
GH-15891 is a backport of this pull request to the 3.8 branch. |
|
Buildbot failure should have been fixed by GH-15902. |
Add unittests for executables with a zipfile appended to test_zipfile, as zipfile.is_zipfile and zipfile.ZipFile work properly on these today.
Adds unittests for executables with a zipfile appended to test_zipfile
as zipfile.is_zipfile and zipfile.ZipFile work properly on these today.
This is being added as a regression test that #5053 would otherwise fail today.
https://bugs.python.org/issue28494