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 the tar Posix option for tarballs #358

Closed
wants to merge 1 commit into from
Closed

Conversation

simo5
Copy link
Contributor

@simo5 simo5 commented Dec 20, 2016

This is necessary to be able to successfully build archives in
environments controlled by an IPA domain which may have large uidNumbers
for user accounts.

https://fedorahosted.org/freeipa/ticket/6418

Signed-off-by: Simo Sorce simo@redhat.com

@pspacek
Copy link
Contributor

pspacek commented Dec 21, 2016

Please add a note why it is necessary. Something like "ustar format we used before could not handle files with > 65535" or so. This will help a lot with Git archaeology.

@pspacek
Copy link
Contributor

pspacek commented Dec 21, 2016

CondACK if the message is corrected.

This is necessary to be able to successfully build archives in
environments controlled by an IPA domain which may have large uidNumbers
for user accounts.

tar-ustar allows UID/GID numbers only up to 2 million and by default a
new IPA installation can assigne UIDs in the billion range.

https://fedorahosted.org/freeipa/ticket/6418

Signed-off-by: Simo Sorce <simo@redhat.com>
@simo5
Copy link
Contributor Author

simo5 commented Dec 21, 2016

Amended

@pspacek pspacek self-requested a review December 21, 2016 13:50
@pspacek pspacek added the ack Pull Request approved, can be merged label Dec 21, 2016
@pspacek
Copy link
Contributor

pspacek commented Dec 21, 2016

Thanks, ACK!

@martbab martbab added the pushed Pull Request has already been pushed label Dec 21, 2016
@martbab
Copy link
Contributor

martbab commented Dec 21, 2016

@martbab martbab closed this Dec 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
3 participants