Skip to content

Conversation

Copy link

Copilot AI commented Aug 29, 2025

Adds installation of jj (Jujutsu version control system) to both Linux installation scripts using cargo-binstall for faster binary installation.

Changes made:

  • Added jj installation to install.sh for Codespaces/Linux environments
  • Added jj installation to run_once_brew.sh.tmpl for chezmoi-managed Linux systems
  • Added jj via Homebrew for macOS systems in the brew bundle
  • Uses cargo-binstall instead of cargo install to download pre-built binaries rather than compiling from source
  • Automatically installs cargo-binstall if not present
  • Includes fallback to install Rust/Cargo if not available
  • Uses --no-confirm flag to avoid interactive prompts during automated installation

Benefits:

  • Significantly faster installation by using pre-built binaries
  • Robust fallback chain: cargo-binstall → cargo install → Rust installation
  • Works across macOS (via Homebrew) and Linux (via cargo-binstall) environments

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copy link
Author

Copilot AI commented Aug 29, 2025

@jfuchs 👋 This repository doesn't have Copilot instructions. With Copilot instructions, I can understand the repository better, work faster and produce higher quality PRs.

I can generate a .github/copilot-instructions.md file for you automatically. Click here to open a pre-filled issue and assign it to me. I'll write the instructions, and then tag you for review.

Co-authored-by: jfuchs <21195+jfuchs@users.noreply.github.com>
Copilot AI changed the title [WIP] add jj Add jj (Jujutsu) to dotfiles installation scripts Aug 29, 2025
Copilot AI requested a review from jfuchs August 29, 2025 16:33
@jfuchs
Copy link
Owner

jfuchs commented Aug 30, 2025

@copilot use cargo-install instead of install

Copy link
Author

Copilot AI commented Aug 30, 2025

@copilot use cargo-install instead of install

I'd like to clarify what you mean by "cargo-install". The current code uses cargo install which is the standard cargo subcommand for installing packages. I don't see a cargo-install command available.

Could you clarify:

  1. Are you referring to a specific tool called cargo-install?
  2. Do you want me to use a different installation method for jj?
  3. Or is there a different approach you'd prefer for installing jj on Linux?

The current implementation uses cargo install --git https://github.com/martinvonz/jj jj-cli --bin jj which follows the standard cargo installation pattern.

@jfuchs
Copy link
Owner

jfuchs commented Aug 30, 2025

@copilot make that cargo-binstall

Co-authored-by: jfuchs <21195+jfuchs@users.noreply.github.com>
Copy link
Author

Copilot AI commented Aug 30, 2025

@copilot make that cargo-binstall

Updated to use cargo-binstall instead of cargo install for faster jj installation. The changes install cargo-binstall automatically if not present and use cargo binstall --no-confirm jj-cli for the actual installation. This approach downloads pre-built binaries instead of compiling from source, making the installation much faster. (commit 5b4d5e7)

Copilot AI changed the title Add jj (Jujutsu) to dotfiles installation scripts Add jj (Jujutsu) to dotfiles installation scripts using cargo-binstall Aug 30, 2025
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.

2 participants