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

Dependency nix needs to be updated #14

Open
Efreak opened this issue Nov 11, 2021 · 1 comment
Open

Dependency nix needs to be updated #14

Efreak opened this issue Nov 11, 2021 · 1 comment

Comments

@Efreak
Copy link

Efreak commented Nov 11, 2021

nix 0.7 is specified as a dependency of tcalc-rustyline. This package doesn't build on Android using rustic 1.56.1.

❯ cargo install tcalc
    Updating crates.io index
  Installing tcalc v1.7.16
   Compiling libc v0.2.107
   Compiling semver v0.1.20
   Compiling cfg-if v0.1.10
   Compiling bitflags v0.4.0
   Compiling void v1.0.2
   Compiling encode_unicode v0.1.3
   Compiling unicode-width v0.1.9
   Compiling rustc_version v0.1.7
   Compiling nix v0.7.0
   Compiling dirs-sys v0.3.6
   Compiling dirs v2.0.2
error[E0518]: attribute should be applied to function or closure
  --> /data/data/com.termux/files/home/.cargo/registry/src/github.com-1ecc6299db9ec823/nix-0.7.0/src/sys/termios.rs:42:5
   |
42 |       #[inline]
   |       ^^^^^^^^^
43 |       #[cfg(all(target_os = "android", not(target_arch = "mips")))]
44 | /     mod android {
45 | |         use libc;
46 | |         use libc::c_int;
47 | |         use super::consts::*;
...  |
90 | |         }
91 | |     }
   | |_____- not a function or closure

For more information about this error, try `rustc --explain E0518`.
error: failed to compile `tcalc v1.7.16`, intermediate artifacts can be found at `/data/data/com.termux/files/usr/tmp/cargo-installT9AStZ`

Caused by:
  could not compile `nix` due to previous error

~/src/cargo took 6s
@dubrowgn
Copy link
Owner

Thanks for the report. I've been thinking about migrating to https://crates.io/crates/reedline, since the keybindings appear to be customizable. This would avoid the need to maintain a fork of rustyline.

Also, sorry for the (very) late response. I somehow missed this originally :(

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