Skip to content

nproc: Minor cleanup#11192

Merged
cakebaker merged 1 commit intouutils:mainfrom
oech3:patch-2
Mar 4, 2026
Merged

nproc: Minor cleanup#11192
cakebaker merged 1 commit intouutils:mainfrom
oech3:patch-2

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Mar 4, 2026

  • Use libc::sysconf at unix.
  • Make funcs oneliner.

@oech3 oech3 marked this pull request as ready for review March 4, 2026 11:41
@github-actions
Copy link

github-actions bot commented Mar 4, 2026

GNU testsuite comparison:

Skip an intermittent issue tests/date/date-locale-hour (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/pr/bounded-memory (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tty/tty-eof (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/rm/isatty is no longer failing!
Congrats! The gnu test tests/dd/no-allocate is now passing!
Congrats! The gnu test tests/tail/tail-n0f is now passing!

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 4, 2026

Merging this PR will degrade performance by 4.79%

❌ 1 regressed benchmark
✅ 301 untouched benchmarks
⏩ 44 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation shuf_input_range[1000000] 84.3 ms 88.5 ms -4.79%

Comparing oech3:patch-2 (3c3eb94) with main (e9ad8b8)

Open in CodSpeed

Footnotes

  1. 44 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@cakebaker cakebaker merged commit e4d943b into uutils:main Mar 4, 2026
161 of 221 checks passed
@cakebaker
Copy link
Contributor

Thanks!

@oech3 oech3 deleted the patch-2 branch March 4, 2026 13:56
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