Skip to content

gh-109980: Fix test_tarfile_vs_tar on macOS #112905

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

Merged
merged 2 commits into from
Dec 10, 2023

Conversation

ronaldoussoren
Copy link
Contributor

@ronaldoussoren ronaldoussoren commented Dec 9, 2023

On recentish macOS versions the system tar
command includes system metadata (ACLs, extended attributes and resource forks) in the tar archive, which
shutil.make_archive will not do. This can cause
spurious test failures.

On recentish macOS versions the system tar
command includes system metadata (ACLs, extended attributes
and resource forks) in the tar archive, which
shutil.make_archive will not do. This can cause
spurious test failures.
@bedevere-app bedevere-app bot added tests Tests in the Lib/test dir awaiting core review labels Dec 9, 2023
@ronaldoussoren ronaldoussoren requested a review from a team December 9, 2023 20:40
@ronaldoussoren ronaldoussoren added needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Dec 9, 2023
@gvanrossum
Copy link
Member

This fixes it for my M3 running Sonoma.

Copy link
Member

@sobolevn sobolevn left a comment

Choose a reason for hiding this comment

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

I've tried this fix and it passes for me (just like the main branch though): m2 sonoma.

@ronaldoussoren ronaldoussoren merged commit dd2ebdf into python:main Dec 10, 2023
@miss-islington-app
Copy link

Thanks @ronaldoussoren for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

@bedevere-app
Copy link

bedevere-app bot commented Dec 10, 2023

GH-112927 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Dec 10, 2023
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 10, 2023
On recentish macOS versions the system tar
command includes system metadata (ACLs, extended attributes
and resource forks) in the tar archive, which
shutil.make_archive will not do. This can cause
spurious test failures.
(cherry picked from commit dd2ebdf)

Co-authored-by: Ronald Oussoren <ronaldoussoren@mac.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 10, 2023
On recentish macOS versions the system tar
command includes system metadata (ACLs, extended attributes
and resource forks) in the tar archive, which
shutil.make_archive will not do. This can cause
spurious test failures.
(cherry picked from commit dd2ebdf)

Co-authored-by: Ronald Oussoren <ronaldoussoren@mac.com>
@bedevere-app
Copy link

bedevere-app bot commented Dec 10, 2023

GH-112928 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Dec 10, 2023
ronaldoussoren added a commit that referenced this pull request Dec 10, 2023
gh-109980: Fix test_tarfile_vs_tar on macOS (GH-112905)

On recentish macOS versions the system tar
command includes system metadata (ACLs, extended attributes
and resource forks) in the tar archive, which
shutil.make_archive will not do. This can cause
spurious test failures.
(cherry picked from commit dd2ebdf)

Co-authored-by: Ronald Oussoren <ronaldoussoren@mac.com>
ronaldoussoren added a commit that referenced this pull request Dec 10, 2023
gh-109980: Fix test_tarfile_vs_tar on macOS (GH-112905)

On recentish macOS versions the system tar
command includes system metadata (ACLs, extended attributes
and resource forks) in the tar archive, which
shutil.make_archive will not do. This can cause
spurious test failures.
(cherry picked from commit dd2ebdf)

Co-authored-by: Ronald Oussoren <ronaldoussoren@mac.com>
@gvanrossum
Copy link
Member

Thanks!

aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
On recentish macOS versions the system tar
command includes system metadata (ACLs, extended attributes
and resource forks) in the tar archive, which
shutil.make_archive will not do. This can cause
spurious test failures.
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
On recentish macOS versions the system tar
command includes system metadata (ACLs, extended attributes
and resource forks) in the tar archive, which
shutil.make_archive will not do. This can cause
spurious test failures.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants