From a41691b5ff88cc19de4e3cd5c70f50f6c132e3b4 Mon Sep 17 00:00:00 2001 From: Rusty Conover Date: Mon, 13 Jan 2025 08:31:03 -0400 Subject: [PATCH] fix: change readme to use stable rust toolchain --- README.md | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/README.md b/README.md index bde48ed..7650154 100644 --- a/README.md +++ b/README.md @@ -302,20 +302,19 @@ For the DuckDB extension to call the Rust code a tool called `cbindgen` is used The headers can be updated by running `make rust_binding_headers`. -#### Building on macOS +#### Building on MacOS X Example setup + build steps for macOS users: ```sh -# Remove rust if previously installed via brew +# Remove rust if previously installed via brew brew uninstall rust # Install rustup + cbindgen # (use rustup to switch versions of Rust without extra fuss) brew install cbindgen rustup -# Install nightly rust toolchain -rustup toolchain install nightly +rustup toolchain install stable # Initialize rustup # Zsh users: customize installation, answer n to "Modify PATH variable?", @@ -323,17 +322,15 @@ rustup toolchain install nightly rustup-init # OPTIONAL step for zsh users: add rust + cargo env setup to zshrc: -echo '. "$HOME/.cargo/env"' >> ~/.zshrc +echo '. "$HOME/.cargo/env"' >> ~/.zshrc -# Use rustc nightly version by default -rustup default nightly +# Use rustc stable version by default +rustup default stable # Build headers make rust_binding_headers -# Using ninja for theorhetical parallel build steps (untested!) -# Set CMAKE_BUILD_PARALLEL_LEVEL to the number of cores on your machine -GEN=ninja CMAKE_BUILD_PARALLEL_LEVEL=8 make +GEN=ninja make ``` ### Build steps