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.

Sign up for GitHub

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

Jump to bottom

Spurious WebRender failure: 'gperf' is missing on your system #69895

Closed
pietroalbini opened this issue Mar 10, 2020 · 1 comment · Fixed by #70626
Closed

Spurious WebRender failure: 'gperf' is missing on your system #69895

pietroalbini opened this issue Mar 10, 2020 · 1 comment · Fixed by #70626
Labels
A-spurious Area: Spurious failures in builds (spuriously == for no apparent reason) C-bug Category: This is a bug. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Comments

@pietroalbini
Copy link
Member

Occasionally the x86_64-gnu-aux builder fails due to a spurious build failure:

make[2]: warning: -jN forced in submake: disabling jobserver mode.
/cargo/registry/src/github.com-1ecc6299db9ec823/servo-fontconfig-sys-4.0.4/missing: line 81: gperf: command not found
WARNING: 'gperf' is missing on your system.
         You might have modified some files without having the proper
         tools for further handling them.  Check the 'README' file, it
         often tells you about the needed prerequisites for installing
         this package.  You may also peek at any GNU archive site, in
         case some other package contains this missing 'gperf' program.
make[4]: *** [fcobjshash.h] Error 1
make[4]: *** Waiting for unfinished jobs....
make[3]: *** [all-recursive] Error 1
make[2]: *** [all] Error 2
make[1]: *** [/checkout/obj/build/ct/webrender/target/debug/build/servo-fontconfig-sys-e3947402e30e86b7/out/libfontconfig.a] Error 2
thread 'main' panicked at 'assertion failed: Command::new("make").env("MAKEFLAGS",
                         env::var("CARGO_MAKEFLAGS").unwrap_or_default()).args(&["-R",
                                                                                 "-f",
                                                                                 "makefile.cargo"]).status().unwrap().success()', /cargo/registry/src/github.com-1ecc6299db9ec823/servo-fontconfig-sys-4.0.4/build.rs:17:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

The WebRender people prepared a fix, but it did not land in https://github.com/servo/webrender yet. Once it lands, we'll need to update cargotest to point to the new commit.

Putting it P-high so we check in on it weekly during the infrastructure team meeting.
@pietroalbini pietroalbini added P-high High priority A-spurious Area: Spurious failures in builds (spuriously == for no apparent reason) T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. C-bug Category: This is a bug. labels Mar 10, 2020
@pietroalbini pietroalbini mentioned this issue Mar 10, 2020
Merged
pietroalbini added a commit to pietroalbini/rust that referenced this issue Mar 11, 2020
@pietroalbini
ci: bump webrender in cargotest
bbafe47 
This bumps the commit of webrender we test to include a fix for a
spurious failure we encountered in CI. See rust-lang#69895 for more context on
the spurious failure.
@pietroalbini pietroalbini mentioned this issue Mar 11, 2020
Closed
@pietroalbini pietroalbini linked a pull request Mar 11, 2020 that will close this issue
ci: bump webrender in cargotest #69928
Closed
Centril added a commit to Centril/rust that referenced this issue Mar 11, 2020
@Centril
Rollup merge of rust-lang#69928 - pietroalbini:bump-cargotest-webrend…
3002bd2 
…er, r=Mark-Simulacrum

ci: bump webrender in cargotest

This bumps the commit of webrender we test to include a fix for a spurious failure we encountered in CI. See rust-lang#69895 for more context on the spurious failure.

The commit we're interested in including is servo/webrender@07439c5. Thanks again @jrmuizel for the quick response on this!

r? @Mark-Simulacrum
@pietroalbini pietroalbini linked a pull request Apr 1, 2020 that will close this issue
cargotest: remove webrender #70626
Merged
@pietroalbini pietroalbini removed the P-high High priority label Apr 1, 2020
@pietroalbini
Copy link
Member Author

Discussed this at the infra meeting, there is a PR removing webrender. De-prioritizing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-spurious Area: Spurious failures in builds (spuriously == for no apparent reason) C-bug Category: This is a bug. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

cargotest: remove webrender pietroalbini/rust
ci: bump webrender in cargotest pietroalbini/rust
1 participant
@pietroalbini