Skip to content

[nodejs] 25.8.0-3: switch to ninja build system#7224

Merged
ziyao233 merged 1 commit into
eweOS:nodejsfrom
Integral-Tech:nodejs
May 16, 2026
Merged

[nodejs] 25.8.0-3: switch to ninja build system#7224
ziyao233 merged 1 commit into
eweOS:nodejsfrom
Integral-Tech:nodejs

Conversation

@Integral-Tech
Copy link
Copy Markdown

Building Node.js using Ninja is faster than using GNU Make.

Building Node.js using Ninja is faster than using GNU Make.
@ziyao233
Copy link
Copy Markdown
Member

Are there any benchmark results demonstrating the improvements?

@Integral-Tech
Copy link
Copy Markdown
Author

Are there any benchmark results demonstrating the improvements?

Before: image

After: image

@ziyao233
Copy link
Copy Markdown
Member

Tasks might be dispatched to workers with different performance levels, so I don't think this comparison makes much sense. I'll give it a try locally.

@ziyao233
Copy link
Copy Markdown
Member

On the same local machine, with the patch,

real    26m12.876s
user    390m34.788s
sys     41m50.966s

without the patch,

real    41m42.511s
user    392m13.776s
sys     40m45.969s

So it does improve the build time, while the user/sys part remains similar, thus I guess it's a concurrency issue.

Anyway, thanks for your contribution!

@github-project-automation github-project-automation Bot moved this from New to Approved in eweOS Package Management May 16, 2026
@ziyao233 ziyao233 merged commit fcda1d9 into eweOS:nodejs May 16, 2026
9 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Done in eweOS Package Management May 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants