diff options
author | bors[bot] <26634292+bors[bot]@users.noreply.github.com> | 2020-07-07 21:36:18 +0100 |
---|---|---|
committer | GitHub <[email protected]> | 2020-07-07 21:36:18 +0100 |
commit | 56ade20380a028026eeb71af2a8a81e37ede7efb (patch) | |
tree | fbf0e24250cc36eaa122904e78cd7cb50fe1c665 /crates/ra_proc_macro_srv/src/cli.rs | |
parent | 980a67f44629ed67a54b603aaf9d015a81d61f7a (diff) | |
parent | f92bfb580780cda02f9ba8a935538f984d8a4c0d (diff) |
Merge #4972
4972: Gzip artifacts r=Veetaha a=Veetaha
[Here is the test release](https://github.com/Veetaha/rust-analyzer/releases/tag/2020-06-21)
Change in size: `~ 25 MB -> ~ 8 MB (gzipped)`
The time to gzip during the dist build takes a somewhat considerable amount of time tho.
Having already compiled artifacts this takes in debug mode:
```
~/dev/rust-analyzer (feat/gzip-binaries) $ time cargo xtask dist
Finished dev [unoptimized] target(s) in 0.06s
Running `target/debug/xtask dist`
> cargo build --manifest-path ./crates/rust-analyzer/Cargo.toml --bin rust-analyzer --release
Finished release [optimized] target(s) in 0.05s
> strip ./target/release/rust-analyzer
real 0m34.331s
user 0m34.245s
sys 0m0.078s
```
In release mode this is much faster:
```
~/dev/rust-analyzer (feat/gzip-binaries) $ time cargo run -p xtask --release -- dist
Finished release [optimized] target(s) in 0.04s
Running `target/release/xtask dist`
> cargo build --manifest-path ./crates/rust-analyzer/Cargo.toml --bin rust-analyzer --release
Finished release [optimized] target(s) in 0.06s
> strip ./target/release/rust-analyzer
real 0m2.401s
```
**[UPD]** adding a profile override for `miniz_oxide` does the thing to ensure good performrance
We might need to notify all other ra plugins' maintainers about the change in our GH releases if we merge this PR, or we could leave uncompressed files along with gzipped for a while until everyone migrates.
Co-authored-by: Veetaha <[email protected]>
Diffstat (limited to 'crates/ra_proc_macro_srv/src/cli.rs')
0 files changed, 0 insertions, 0 deletions