Skip to content

Conversation

@sylvestre
Copy link
Contributor

This will fix the parsing for old autoconf
Closes: #8880

@codspeed-hq
Copy link

codspeed-hq bot commented Oct 16, 2025

CodSpeed Performance Report

Merging #8921 will degrade performances by 9.44%

Comparing sylvestre:help-path (3593603) with main (ca8eb4d)

Summary

⚡ 1 improvement
❌ 4 regressions
✅ 101 untouched
⏩ 73 skipped1

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

Benchmarks breakdown

Benchmark BASE HEAD Change
b64_decode_ignore_garbage_synthetic 145.1 µs 149.1 µs -2.64%
ls_recursive_long_all_balanced_tree[(6, 4, 15)] 141.3 ms 138.4 ms +2.09%
mv_force_overwrite 118.7 ms 121.5 ms -2.23%
mv_single_file 118.6 ms 130.9 ms -9.44%
rm_single_file 103 ms 106.5 ms -3.36%

Footnotes

  1. 73 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.

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/usage_vs_getopt (fails in this run but passes in the 'main' branch)

@sylvestre sylvestre requested a review from cakebaker October 21, 2025 20:07
@sylvestre sylvestre marked this pull request as ready for review October 21, 2025 20:07
@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/timeout/timeout (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/install/install-Z-selinux is no longer failing!

@sylvestre sylvestre force-pushed the help-path branch 2 times, most recently from 985041f to 7de1ae0 Compare October 22, 2025 21:03
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

This will fix the parsing for old autoconf
Closes: uutils#8880
@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/timeout/timeout (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

Co-authored-by: Daniel Hofstetter <daniel.hofstetter@42dh.com>
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/timeout/timeout (passes in this run but fails in the 'main' branch)

@cakebaker cakebaker merged commit 3d6b0b2 into uutils:main Oct 23, 2025
120 of 121 checks passed
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.

old configure (with autoconf ≥ 2.72) scripts don't detect mkdir -p as race-free

2 participants