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

Add generation of XZ-compressed tarballs #57

Merged
merged 2 commits into from Apr 27, 2017

Conversation

Projects
None yet
2 participants
@ranma42
Contributor

ranma42 commented Apr 11, 2017

Compress the tarballs with both gzip (for backward compatibility) and
xz (to get better compression).

This is part of the preliminary work for rust-lang/rust#21724

Add generation of XZ-compressed tarballs
Compress the tarballs with both gzip (for backward compatibility) and
xz (to get better compression).
@ranma42

This comment has been minimized.

Show comment
Hide comment
@ranma42

ranma42 Apr 11, 2017

Contributor

The compression can be performed using 7z or xz. AppVeyor provides 7z by default and I think it might be easier to install packages on Linux/Mac (i.e. the TravisCI environments) with apt install p7zip-full or brew install p7zip. Is this ok?

Contributor

ranma42 commented Apr 11, 2017

The compression can be performed using 7z or xz. AppVeyor provides 7z by default and I think it might be easier to install packages on Linux/Mac (i.e. the TravisCI environments) with apt install p7zip-full or brew install p7zip. Is this ok?

Fix testing by installing 7z
provided by the `p7zip-full` package.
@ranma42

This comment has been minimized.

Show comment
Hide comment
@ranma42

ranma42 Apr 27, 2017

Contributor

Unless this PR gets merged shortly, it might make sense to integrate it in the Rust-based rust-installer rust-lang/rust#41569

Contributor

ranma42 commented Apr 27, 2017

Unless this PR gets merged shortly, it might make sense to integrate it in the Rust-based rust-installer rust-lang/rust#41569

@brson brson merged commit d7855d6 into rust-lang:master Apr 27, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@brson

This comment has been minimized.

Show comment
Hide comment
@brson

brson Apr 27, 2017

Contributor

Looks right to me. Thanks @ranma42. Next step is to update the submodule in rust-lang/rust, and update the CI configuration to make 7zip available everywhere.

Contributor

brson commented Apr 27, 2017

Looks right to me. Thanks @ranma42. Next step is to update the submodule in rust-lang/rust, and update the CI configuration to make 7zip available everywhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment