nushell / nushell.github.io

Nushell's main website, blog, book, and more
https://www.nushell.sh/book/
MIT License
162 stars 401 forks source link

Update installation instructions with `cargo` #1460

Closed Garbaz closed 1 week ago

Garbaz commented 1 week ago

Hi,

This is only a small change.

I have removed the paragraph about including the dataframes feature, as it no longer exists (I presume it's just enabled by default?). At least the command cargo install nu --locked --features=dataframe given in the book fails with:

error: failed to compile `nu v0.95.0`, intermediate artifacts can be found at `/tmp/cargo-install0x8JXF`.
To reuse those artifacts with a future compilation, set the environment variable `CARGO_TARGET_DIR` to that path.

Caused by:
  none of the selected packages contains these features: dataframe

I also added --locked to the remaining cargo install command.

fdncred commented 1 week ago

We traditionally don't recommend using --locked. We use that as a fallback mechanism if an upstream crate is misbehaving. Please remove that and we can land this.

Garbaz commented 1 week ago

:+1: Okay, done.

fdncred commented 1 week ago

Thanks