Skip to content

cargo chef fails with profile ci is missing an inherits directive #642

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

Closed
Extarys opened this issue Jun 27, 2022 · 4 comments
Closed

cargo chef fails with profile ci is missing an inherits directive #642

Extarys opened this issue Jun 27, 2022 · 4 comments

Comments

@Extarys
Copy link
Contributor

Extarys commented Jun 27, 2022

With latest pull from av1an/master, I can't no longer build the docker image.

Step 13/27 : RUN cargo chef cook --release --recipe-path recipe.json
[...]
  Downloaded rayon-core v1.9.1
  Downloaded pin-project-lite v0.2.8
  Downloaded num_threads v0.1.5
  Downloaded crossbeam-utils v0.8.8
  Downloaded bindgen v0.59.2
  Downloaded cc v1.0.73
  Downloaded bytes v1.1.0
  Downloaded arrayvec v0.7.2
  Downloaded unicode-normalization v0.1.19
  Downloaded idna v0.2.3
  Downloaded indexmap v1.8.0
  Downloaded git2 v0.14.2
  Downloaded flexi_logger v0.22.3
  Downloaded ffmpeg-sys-next v5.0.1
  Downloaded anyhow v1.0.56
  Downloaded libc v0.2.121
  Downloaded libgit2-sys v0.13.2+1.4.2
  Downloaded libz-sys v1.1.5
  Downloaded plotters v0.3.1
error: profile `ci` is missing an `inherits` directive (`inherits` is required for all profiles except `dev` or `release`)
thread 'main' panicked at 'Exited with status code: 101', /root/.cargo/registry/src/github.jpy.wang-1ecc6299db9ec823/cargo-chef-0.1.36/src/recipe.rs:148:27
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
The command '/bin/sh -c cargo chef cook --release --recipe-path recipe.json' returned a non-zero code: 101
@redzic
Copy link
Collaborator

redzic commented Jun 28, 2022

Possibly a bug in cargo-chef? Removing the ci profile from Cargo.toml fixes the docker build for me, but unfortunately I don't think that's feasible since the scene detection on the debug profile is unbearably slow, and we wanted to still enable overflow checks to catch errors in CI.

Also strange because the ci profile isn't actually missing an inherits directive.

@FreezyLemon
Copy link
Contributor

Fixing the cargo-chef version via RUN cargo install cargo-chef --version 0.1.35 works in my local docker containers to build again. This might be a viable workaround until it is fixed upstream. Or has it been tried already?

@FreezyLemon
Copy link
Contributor

Update: This has been fixed upstream. The issue was a bug in a dependency of cargo-chef. Reverting #643 should work. (Oh but please keep the fixed pacman -Syu at the top)

@Extarys
Copy link
Contributor Author

Extarys commented Jul 4, 2022

Should be fixed.

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

No branches or pull requests

3 participants