Skip to content

CI: use parallel make whenever possible - #7389

Closed
neteler wants to merge 1 commit into
OSGeo:mainfrom
neteler:gh_workflows_parallel_make_jobs
Closed

neteler wants to merge 1 commit into
OSGeo:mainfrom
neteler:gh_workflows_parallel_make_jobs

Conversation

@neteler

@neteler neteler commented May 10, 2026

Copy link
Copy Markdown
Member

This PR introduces where missing make -j$(nproc) to make use of the multi-core runners, see https://docs.github.com/en/actions/reference/runners/github-hosted-runners#standard-github-hosted-runners-for-public-repositories

For ubuntu based runners, it is at present 4 Processor (CPU).

Hence, running CI time should be significantly cut.

This PR introduces where missing `make -j$(nproc)` to make use of the multi-core runners, see
<https://docs.github.com/en/actions/reference/runners/github-hosted-runners#standard-github-hosted-runners-for-public-repositories>

For ubuntu based runners, it is at present 4 Processor (CPU).

Hence, running CI time should be significantly cut.
@neteler neteler added this to the 8.6.0 milestone May 10, 2026
@neteler neteler self-assigned this May 10, 2026
@neteler neteler added CI Continuous integration backport to 8.5 PR needs to be backported to release branch 8.5 labels May 10, 2026
@echoix

echoix commented May 10, 2026

Copy link
Copy Markdown
Member

They should already be using it, because of lines like

- name: Set number of cores for compilation
run: |
echo "MAKEFLAGS=-j$(nproc)" >> "${GITHUB_ENV}"

@neteler

neteler commented May 10, 2026

Copy link
Copy Markdown
Member Author

MAKEFLAGS

Is this variable actually used anywhere?

@neteler neteler removed the backport to 8.5 PR needs to be backported to release branch 8.5 label May 10, 2026
@neteler neteler removed this from the 8.6.0 milestone May 10, 2026
@neteler

neteler commented May 10, 2026

Copy link
Copy Markdown
Member Author

Okay, found it in the make documentation. Sorry for the noise, closing.

@neteler neteler closed this May 10, 2026
@echoix

echoix commented May 10, 2026

Copy link
Copy Markdown
Member

MAKEFLAGS

Is this variable actually used anywhere?

Yes, used directly by make. There's also GNUMAKEFLAGS that is used if I remember well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants