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

Use file_name and link_name from PAX header #128

Merged
merged 1 commit into from
Jun 6, 2023
Merged

Use file_name and link_name from PAX header #128

merged 1 commit into from
Jun 6, 2023

Conversation

reynir
Copy link
Member

@reynir reynir commented Jun 5, 2023

Fixes #126.

This is back ported from #127 which will be a breaking change (and may take some time to finish).

@reynir reynir marked this pull request as ready for review June 5, 2023 13:16
@reynir reynir changed the base branch from main to v2.5.x June 5, 2023 13:24
@reynir reynir merged commit 4a7766c into v2.5.x Jun 6, 2023
@reynir reynir deleted the fix-126 branch June 6, 2023 07:42
reynir added a commit to reynir/opam-repository that referenced this pull request Jun 6, 2023
CHANGES:

- File names and link names are used from PAX headers when parsing (reported by @gravicappa, fixed in mirage/ocaml-tar#128 by @reynir)
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.

Tar.Header.unmarshal ignores extended.path and extended.link_path.
1 participant