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

[plan-build] Extract tar with ownership of current user. #3972

Merged
merged 1 commit into from
Nov 1, 2017

Conversation

fnichol
Copy link
Collaborator

@fnichol fnichol commented Nov 1, 2017

This change will ignore user and group metadata in source tarballs when
extracting them in do_unpack() or when using unpack_file(). Trying
to set unknown uid and gid numbers to files can cause issues and
shouldn't be required when building the software.

Signed-off-by: Fletcher Nichol fnichol@nichol.ca

This change will ignore user and group metadata in source tarballs when
extracting them in `do_unpack()` or when using `unpack_file()`. Trying
to set unknown uid and gid numbers to files can cause issues and
shouldn't be required when building the software.

Signed-off-by: Fletcher Nichol <fnichol@nichol.ca>
@thesentinels
Copy link
Contributor

Thanks for the pull request! Here is what will happen next:

  1. Your PR will be reviewed by the maintainers
  2. If everything looks good, one of them will approve it, and your PR will be merged.

Thank you for contributing!

@reset reset merged commit d7d6919 into master Nov 1, 2017
@fnichol fnichol deleted the fnichol/plan-build-tar-unpacking-ownership branch November 1, 2017 04:12
@eeyun eeyun added I-linux and removed P-linux labels Mar 6, 2018
@christophermaier christophermaier added Platform: Linux Deals with Linux-specific behavior Type: Chore Issues for general code and infrastructure maintenance and removed I-linux labels Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Focus :Plan Build Platform: Linux Deals with Linux-specific behavior Type: Chore Issues for general code and infrastructure maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants