Skip to content

scripts: add -j/--jobs and MOQX_BUILD_JOBS to control build parallelism - #523

Merged
afrind merged 1 commit into
mainfrom
build-jobs-flag
Jul 27, 2026
Merged

scripts: add -j/--jobs and MOQX_BUILD_JOBS to control build parallelism#523
afrind merged 1 commit into
mainfrom
build-jobs-flag

Conversation

@afrind

@afrind afrind commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Both build.sh and setup-deps-standalone.sh hardcoded nproc for the cmake --build job count. distcc wants far more jobs than the local core count, so add an explicit override with precedence: -j flag, then MOQX_BUILD_JOBS, then nproc.

Also loosen the optional-SHA lookahead in --from-release/--from-source from "$1" != --* to "$1" != -* so a following -j isn't consumed as a SHA.


This change is Reviewable

Both build.sh and setup-deps-standalone.sh hardcoded nproc for the
cmake --build job count. distcc wants far more jobs than the local core
count, so add an explicit override with precedence: -j flag, then
MOQX_BUILD_JOBS, then nproc.

Also loosen the optional-SHA lookahead in --from-release/--from-source
from "$1" != --* to "$1" != -* so a following -j isn't consumed as a SHA.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@afrind
afrind requested review from gmarzot and michalhosna July 27, 2026 21:46

@gmarzot gmarzot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@gmarzot reviewed 2 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on michalhosna).

@afrind
afrind merged commit a80632e into main Jul 27, 2026
16 checks passed
@afrind
afrind deleted the build-jobs-flag branch July 27, 2026 22:34
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