Skip to content

Improvement: add flag to limit build jobs#640

Open
daleshort wants to merge 1 commit intodiodeinc:mainfrom
daleshort:main
Open

Improvement: add flag to limit build jobs#640
daleshort wants to merge 1 commit intodiodeinc:mainfrom
daleshort:main

Conversation

@daleshort
Copy link
Copy Markdown

@daleshort daleshort commented Mar 16, 2026

Adds an optional flag to the install script to optionally limit the number of parallel builds with cargo. I had difficulty getting this to install on a M1 Macbook air (8GB memory) normally and was getting memory errors ("SIGBUS: access to undefined memory") but passing -j 2 had no issues.


Note

Low Risk
Low risk: changes are confined to install.sh, adding simple argument parsing and passing -j to cargo install to reduce parallelism; main risk is minor usability/compatibility issues in the script's new CLI handling.

Overview
install.sh now accepts an optional -j N argument, validates it as a positive integer, and forwards it to cargo install to limit parallel build threads.

The script builds the cargo command dynamically and improves failure guidance by suggesting retrying with ./install.sh -j 1 when builds fail due to memory pressure.

Written by Cursor Bugbot for commit 46a1898. This will update automatically on new commits. Configure here.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is kicking off a free cloud agent to fix this issue. This run is complimentary, but you can enable autofix for all future PRs in the Cursor dashboard.

Comment thread install.sh
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.

1 participant