Skip to content

nproc: process space in OMP_NUM_THREADS#10973

Merged
cakebaker merged 2 commits intouutils:mainfrom
cuiweixie:bug_nproc
Feb 16, 2026
Merged

nproc: process space in OMP_NUM_THREADS#10973
cakebaker merged 2 commits intouutils:mainfrom
cuiweixie:bug_nproc

Conversation

@cuiweixie
Copy link
Contributor

in GNU nproc:

OMP_NUM_THREADS="4 ,8" nproc

output:

4

@cakebaker
Copy link
Contributor

Can you please add a test to test_nproc.rs to ensure we don't regress in the future? Thanks.

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/seq/seq-epipe. tests/seq/seq-epipe is passing on 'main'. Maybe you have to rebase?
Skipping an intermittent issue tests/pr/bounded-memory (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tail/follow-name (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/tail/tail-n0f is no longer failing!
Congrats! The gnu test tests/basenc/bounded-memory is now passing!
Congrats! The gnu test tests/rm/many-dir-entries-vs-OOM is now passing!

@cuiweixie
Copy link
Contributor Author

Can you please add a test to test_nproc.rs to ensure we don't regress in the future? Thanks.

Done

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/misc/io-errors. tests/misc/io-errors is passing on 'main'. Maybe you have to rebase?
Congrats! The gnu test tests/tail/tail-n0f is no longer failing!
Congrats! The gnu test tests/basenc/bounded-memory is now passing!
Congrats! The gnu test tests/rm/many-dir-entries-vs-OOM is now passing!

@cakebaker cakebaker merged commit e93361b into uutils:main Feb 16, 2026
155 of 157 checks passed
@cakebaker
Copy link
Contributor

Thanks for your PR!

abendrothj pushed a commit to abendrothj/coreutils that referenced this pull request Feb 17, 2026
* nproc: process space in OMP_NUM_THREADS

* feat: add unittest
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