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

Failed to build with rustc 1.35.0 #18

Closed
yukinarit opened this issue Jun 7, 2019 · 3 comments
Closed

Failed to build with rustc 1.35.0 #18

yukinarit opened this issue Jun 7, 2019 · 3 comments

Comments

@yukinarit
Copy link

Env

$ cat /etc/system-release
CentOS Linux release 7.4.1708 (Core)

$ rustc --version
rustc 1.35.0 (3c235d560 2019-05-20)

Command

$ cargo install procs

   Compiling console v0.7.5
   Compiling procfs v0.5.2
   Compiling cookie v0.2.5
   Compiling structopt v0.2.17
   Compiling hyper v0.9.18
   Compiling palaver v0.2.1
error[E0308]: mismatched types
  --> /home/AD/yukinari.tani/.cargo/registry/src/github.com-1ecc6299db9ec823/palaver-0.2.1/src/valgrind.rs:15:39
   |
15 |     let err = unsafe { libc::getrlimit64(resource, &mut rlim) };
   |                                          ^^^^^^^^ expected u32, found i32

error[E0308]: mismatched types
  --> /home/AD/yukinari.tani/.cargo/registry/src/github.com-1ecc6299db9ec823/palaver-0.2.1/src/valgrind.rs:33:23
   |
33 |     let rlim = getrlimit(libc::RLIMIT_NOFILE).unwrap();
   |                          ^^^^^^^^^^^^^^^^^^^ expected i32, found u32

error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0308`.
error: Could not compile `palaver`.
warning: build failed, waiting for other jobs to finish...
error: failed to compile `procs v0.8.5`, intermediate artifacts can be found at `/tmp/cargo-installot988m`

Caused by:
  build failed
@dalance
Copy link
Owner

dalance commented Jun 7, 2019

Thanks. I send PR to palaver.
alecmocatta/palaver#15

@dalance
Copy link
Owner

dalance commented Jun 11, 2019

I changed the dependency crate from palaver to process_path because almost all function in palaver were not used.
I released v0.8.6 and it can be cargo install.

@yukinarit
Copy link
Author

Success! Thank you!

$ cargo install procs

   Installed package `procs v0.8.6` (executable `procs`)

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