[BUGFIX] rust-sccache: update to 0.2.15, enable feature=dist-server for x86_64*

[Logarithmus]

Have the results of the proposed changes been tested?

• I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
• I generally don't use the affected packages but briefly tested this PR

[ericonr]

Should be able to remove the cargo update command, since we are now on OpenSSL.

I don't think this needs a sub package, though.

[Logarithmus]

running 1 test
test test_rust_cargo ... FAILED

failures:

---- test_rust_cargo stdout ----
thread 'test_rust_cargo' panicked at 'Failed to spawn "/usr/bin/cargo" "clean": No such file or directory (os error 2)', /host/cargo/registry/src/github.com-1ecc6299db9ec823/assert_cmd-1.0.2/src/assert.rs:63:17
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Hmm...

[Logarithmus]

Should be able to remove the cargo update command, since we are now on OpenSSL.

I don't think this needs a sub package, though.

It definitely needs a subpackage, because most users don't use distributed builds functionality. They just use local and/or S3 cache to speed up repeated builds, for which sccache binary is sufficient. And sccache-dist binary is about 9 MiB, which is quite a lot IMHO, considering that Void aims to be minimal/bloat-free distro.

[Logarithmus]

Turns out sccache-dist supports x86_64 only. So instead of subpackage I decided to create a separate template for it.

[Logarithmus]

@ericonr sccache-0.2.13 is buggy and doesn't work. I always get this error when I run cargo build:

Caused by:
  process didn't exit successfully: `/usr/bin/sccache rustc - --crate-name ___ --print=file-names --crate-type bin --crate-type rlib --crate-type dylib --crate-type cdylib --crate-type staticlib --crate-type proc-macro --print=sysroot --print=cfg` (exit code: 2)
  --- stderr
  error: failed to execute compile
  caused by: Failed to send data to or receive data from server
  caused by: Failed to read response header
  caused by: failed to fill whole buffer

More info: mozilla/sccache#887, mozilla/sccache#887 (comment)
As you can see, version 0.2.13 is unusable. I noticed that and thus filed this PR back in March. Unfortunately, you don't seem interested in merging it, so I decided to provide some proofs that this change is very important.

I've also managed to add sccache support to xbps-src, just like ccache. Obviously, it doesn't work on sccache-0.2.13, so this PR is a blocker.

[ericonr]

Please don't make separate packages, the rust-std package is 117MB compressed, and anyone using this has one (or more) rust-std installed (either via void packages or rustup).

[Logarithmus]

Please don't make separate packages, the rust-std package is 117MB compressed, and anyone using this has one (or more) rust-std installed (either via void packages or rustup).

sccache supports x86_64, i686, arm* & aarch64, and sccache-dist supports only x86_64. How would you handle that in a single package? By checking $XBPS_TARGET_MACHINE and calling cargo build --release manually in template?
IMHO making 2 packages is way simpler & cleaner.

[ericonr]

Just conditionally add --features dist-server, no?

[Logarithmus]

@ericonr seems it builds fine, though during build of x86_64-musl there were some issues with Internet connection on CI server.

[Logarithmus]

@ericonr seems like cargo 0.54.0 has some issue with HTTP/2 and thus fails to work inside Github Actions. The issue has its roots in curl.
More info: rust-lang/cargo#9697, rust-lang/crates.io#3782 (comment), rust-lang/cargo#9695, curl/curl#7400

I really hope rust-lang/cargo#9695 will be included into rust 1.54, otherwise we will have to wait for another 6 weeks...

[redasamik]

بتاريخ الثلاثاء، 27 يوليو 2021، جاء من Artur Sinila < ***@***.***>:

…

@ericonr <https://github.com/ericonr> seems like cargo 0.54.0 has some issue with HTTP/2 and thus fails to work inside Github Actions. The issue has its roots in curl. More info: rust-lang/cargo#9697 <rust-lang/cargo#9697>, rust-lang/crates.io#3782 (comment) <rust-lang/crates.io#3782 (comment)>, rust-lang/cargo#9695 <rust-lang/cargo#9695>, curl/curl#7400 <curl/curl#7400> — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#29756 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AOW7T7HCFPHMAVYINO2E5G3TZYGLXANCNFSM4Z2CBFQA> .

-- reda

[github-actions]

Pull Requests become stale 90 days after last activity and are closed 14 days after that. If this pull request is still relevant bump it or assign it.