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.

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

cargo install cargo-web fails to compile/install #400

Closed
mcrosson opened this issue Dec 30, 2018 · 1 comment
Closed

cargo install cargo-web fails to compile/install #400

mcrosson opened this issue Dec 30, 2018 · 1 comment

Comments

@mcrosson
Copy link
Contributor

Plume version (and/or commit):

With the current Dockerfile cargo install cargo-web fails with the following error. The "fix" I found is to change the COPY Cargo.toml Cargo.lock ./ line to COPY . . and remove the subsequent copy from the Dockerfile.

   Compiling backtrace v0.3.13
error[E0658]: scoped lint `clippy::needless_pass_by_value` is experimental (see issue #44690)
  --> /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/pest-2.1.0/src/error.rs:97:13
   |
97 |     #[allow(clippy::needless_pass_by_value)]
   |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0658]: scoped lint `clippy::needless_pass_by_value` is experimental (see issue #44690)
   --> /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/pest-2.1.0/src/error.rs:137:13
    |
137 |     #[allow(clippy::needless_pass_by_value)]
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0658]: scoped lint `clippy::new_ret_no_self` is experimental (see issue #44690)
   --> /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/pest-2.1.0/src/parser_state.rs:114:13
    |
114 |     #[allow(clippy::new_ret_no_self)]
    |             ^^^^^^^^^^^^^^^^^^^^^^^

error[E0658]: scoped lint `clippy::new_ret_no_self` is experimental (see issue #44690)
  --> /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/pest-2.1.0/src/position.rs:52:13
   |
52 |     #[allow(clippy::new_ret_no_self)]
   |             ^^^^^^^^^^^^^^^^^^^^^^^

error[E0658]: scoped lint `clippy::new_ret_no_self` is experimental (see issue #44690)
  --> /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/pest-2.1.0/src/span.rs:56:13
   |
56 |     #[allow(clippy::new_ret_no_self)]
   |             ^^^^^^^^^^^^^^^^^^^^^^^

error[E0658]: scoped lint `clippy::all` is experimental (see issue #44690)
 --> /usr/local/cargo/registry/src/github.com-1ecc6299db9ec823/pest-2.1.0/src/unicode/mod.rs:6:10
  |
6 | #![allow(clippy::all)]
  |          ^^^^^^^^^^^

   Compiling base64 v0.9.3
   Compiling block-padding v0.1.2
   Compiling humantime v1.2.0
   Compiling regex-syntax v0.6.4
   Compiling walkdir v2.2.7
   Compiling textwrap v0.10.0
   Compiling tokio-executor v0.1.5
   Compiling smallvec v0.6.7
   Compiling want v0.0.6
error: aborting due to 6 previous errors

For more information about this error, try `rustc --explain E0658`.
error: Could not compile `pest`.
warning: build failed, waiting for other jobs to finish...
trinity-1686a pushed a commit that referenced this issue Dec 31, 2018
* Update plume-front build to use a shell script that adds the necessary dependencies on arm (x86 left untouched inside the script)

* Fix cleanup portion of plume-front.sh

* Break wasm dependencies for arm out so they are run before the main builds to improve layer caching on subsequent docker builds

* Fixup dockerfile so wasm-deps can run properly and with fewer assumptions

* Move new scripts to script/ folder

* Remove saving of path as it's not actually not necessary

* Add rust-toolchain to early copy so it pulls the right rust release (addresses #400)
@trinity-1686a
Copy link
Contributor

Fixed by #402

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

2 participants