Skip to content
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

Archive-Tar sync from CPAN with modification #21691

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

jkeenan
Copy link
Contributor

@jkeenan jkeenan commented Dec 5, 2023

No description provided.

3.02  13/04/2023 (Manfred Stock)
- Test block sizes up to (2**31 - 1) bytes only (fix for 32bit perls)
- Don't match on message from exception in symlink test
- Improve formatting of $Archive::Tar::RESOLVE_SYMLINK documentation
To indicate that we are including a patched version of Archive-Tar-3.02
in core, we increment $VERSION in its *.pm files and in %Modules within
Porting/Maintainers.pl.

We're excluding t/90_symlink.t from shipping with core until such time
as (a) we know whether its unit tests are for "developers" only or for
general automated testing; and (b) we get a better 'skip_all' condition.

Fixes GH Perl#21402
Copy link
Contributor

@leonerd leonerd left a comment

Choose a reason for hiding this comment

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

LGTM

@rwp0
Copy link
Contributor

rwp0 commented Dec 7, 2023

Fixes #21402

@jkeenan
Copy link
Contributor Author

jkeenan commented Dec 8, 2023

@haarg, does this approach look okay to you (at least as a temporary measure)?

@jkeenan jkeenan self-assigned this Dec 9, 2023
@jkeenan jkeenan merged commit 54f925e into Perl:blead Dec 12, 2023
29 checks passed
@jkeenan jkeenan deleted the archive-tar-sync-20231105 branch December 12, 2023 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants