Skip to content

Fix MesonNinja easyblock to avoid ninja using all CPUs when only 1 is requested#4023

Merged
boegel merged 2 commits intoeasybuilders:developfrom
Crivella:fix-mesonninja_parallel
Dec 15, 2025
Merged

Fix MesonNinja easyblock to avoid ninja using all CPUs when only 1 is requested#4023
boegel merged 2 commits intoeasybuilders:developfrom
Crivella:fix-mesonninja_parallel

Conversation

@Crivella
Copy link
Copy Markdown
Contributor

Running ninja with no -j option will result in it using all available CPUs, which is the opposite of what one would want when running with max_parallel = 1

@boegel boegel changed the title Fix ninja using all CPUs when only 1 is requested Fix MesonNinja easyblock to avoid ninja using all CPUs when only 1 is requested Dec 15, 2025
Copy link
Copy Markdown
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel added this to the next release (5.2.0) milestone Dec 15, 2025
@boegel
Copy link
Copy Markdown
Member

boegel commented Dec 15, 2025

Test report by @boegel

Overview of tested easyconfigs (in order)

  • SUCCESS Xvfb-21.1.9-GCCcore-13.2.0.eb

  • SUCCESS HarfBuzz-11.4.1-GCCcore-14.3.0.eb

  • SUCCESS Pango-1.57.0-GCCcore-14.3.0.eb

Build succeeded for 3 out of 3 (total: 1 hour 59 mins 39 secs) (3 easyconfigs in total)
node4245.shinx.os - Linux RHEL 9.6, x86_64, AMD EPYC 9654 96-Core Processor (zen4), Python 3.9.21
See https://gist.github.com/boegel/b107c067df29a5d0c5d6f5a3b86c24cb for a full test report.

@boegel boegel merged commit fa75dbe into easybuilders:develop Dec 15, 2025
22 checks passed
@Crivella Crivella deleted the fix-mesonninja_parallel branch December 15, 2025 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants