Skip to content

Improve install guidance for external Cargo target dirs#55

Open
dzianisv wants to merge 1 commit into
mainfrom
fix/install-external-target-dir
Open

Improve install guidance for external Cargo target dirs#55
dzianisv wants to merge 1 commit into
mainfrom
fix/install-external-target-dir

Conversation

@dzianisv
Copy link
Copy Markdown
Owner

Summary

  • make the installer check free space on the filesystem backing CARGO_TARGET_DIR
  • improve low-space help text so it explicitly suggests moving CARGO_TARGET_DIR to another disk
  • document the external-target install pattern and cover it in the installer shell tests

Testing

  • ./scripts/install/test-source-build-install.sh
  • sh -n install.sh scripts/install/test-source-build-install.sh
  • CODEX_INSTALL_DIR=$HOME/.cargo/bin CARGO_TARGET_DIR=/Volumes/Dzianis-3/codex-target TMPDIR=/Volumes/Dzianis-3/codex-tmp CODEX_BUILD_JOBS=1 ./install.sh

Closes #54

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

Successfully merging this pull request may close these issues.

Installer should surface external Cargo target usage when local disk is tight

1 participant