-
-
Notifications
You must be signed in to change notification settings - Fork 634
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add an --all-cores command line option to scons #16943
Conversation
WalkthroughThe recent updates enhance the build process by modifying the SCons command-line parameters across several components. The Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
See test results for failed build of commit deb3b7bb5e |
This feature is really good, but it doesn't showcase its performance on AppVeyor. Will it be faster using GitHub Actions? |
See test results for failed build of commit 5d74e8c733 |
The primary reason why this is not used on appveyor is because it scrambles the output. |
Note that you can set this to be the case always by adding PS D:\projects\nvda> echo $env:SCONSFLAGS
-j 16 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this.
Co-authored-by: Sascha Cowley <16543535+SaschaCowley@users.noreply.github.com>
Link to issue number:
Fixup for #16868
Summary of the issue:
As part of #16868, i intended to implement support to pass a number of 0 cores to scons (e.g.
scons source -j0
) to automatically pick all available cores. It turns out that you cant override an option withSetOption
when set on the command line.Description of user facing changes
None, build system related.
Description of development approach
Added an
--all-cores
parameter that will pick all available cores.Testing strategy:
Ensured that the build was marvelously faster when using the
--all-cores
parameter.Known issues with pull request:
None known
Code Review Checklist:
Summary by CodeRabbit
New Features
--all-cores
for the build process, allowing users to utilize all available CPU cores for improved performance.--all-cores
option in thescons
build tool.Bug Fixes
-j0
to--all-cores
.Documentation
--all-cores
parameter.