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

tools.sh: directly decompress using tar #324

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Gcenx
Copy link
Contributor

@Gcenx Gcenx commented Jan 17, 2022

Modern tar directly supports compressing and decompressing these formats.

Modern tar directly supports compressing and decompressing these formats.
@tpoechtrager
Copy link
Owner

The reason behind this is backwards compatibility. Modern tar isn't available everywhere.

@Gcenx
Copy link
Contributor Author

Gcenx commented Jan 17, 2022

Is there any currently supported version of Linux/unix that ships and old version of tar that doesn’t support this?

The reason I bring it up is gnu tar 1.22 added support for this and that releases in 2009. Mac OS X 10.9 and later ships BSD tar that also supports this.

As the desire is to keep backwards comparability could this extraction be altered to allow the tar methods first then fallback to the current methods?

This is part of an attempt to shrink down the requirements for mac, this change and a couple of others gets the requirements down to clang-3.9Or later and cmake

@tpoechtrager
Copy link
Owner

Using only tar didn't work on NetBSD back in 2014 -> 5f159c9. Don't know about now.

As the desire is to keep backwards comparability could this extraction be altered to allow the tar methods first then fallback to the current methods?

Sure. You can update this PR if you want to.

@Gcenx
Copy link
Contributor Author

Gcenx commented Feb 1, 2022

Sorry gotten sidetracked by other things.

Not sure I'll get chance to test on NetBSD to confirm if it's still shipping an ancient version of tar, after confirming the macOS related modifications I'll update this accordingly or just role it into another PR.

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.

2 participants