Skip to content

Conversation

@oech3
Copy link
Contributor

@oech3 oech3 commented Jan 15, 2026

Fixes #10244
Also covers some (but no all of) use case for symlinking binary to /bin/false.

@oech3 oech3 marked this pull request as draft January 15, 2026 08:31
@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/env/env. tests/env/env is passing on 'main'. Maybe you have to rebase?

@oech3 oech3 force-pushed the core-1 branch 3 times, most recently from e9b9baa to ddd1f27 Compare January 15, 2026 08:55
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 15, 2026

CodSpeed Performance Report

Merging this PR will improve performance by 6.49%

Comparing oech3:core-1 (88e1448) with main (5fd3459)

Summary

⚡ 2 improved benchmarks
✅ 280 untouched benchmarks
⏩ 38 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Memory du_deep_tree[(100, 3)] 155.8 KB 146.3 KB +6.49%
Memory numfmt_large_numbers_si[10000] 4.9 MB 4.7 MB +4.01%

Footnotes

  1. 38 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:

GNU test failed: tests/env/env. tests/env/env is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/env/env. tests/env/env is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/shuf/shuf-reservoir (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/sort/sort-stale-thread-mem (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/env/env. tests/env/env is passing on 'main'. Maybe you have to rebase?

@oech3 oech3 marked this pull request as ready for review January 15, 2026 10:43
Some(OsString::from(util))
} else {
// unmatched binary name => regard as multi-binary container and advance argument list
} else if binary_as_util.ends_with("coreutils") || binary_as_util.ends_with("box") {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't like to have .ends_with("box") in the binary just for test suite. Should be covered by shebang wrapper (at different PR).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shebang does not work...

@oech3 oech3 force-pushed the core-1 branch 2 times, most recently from 572a08f to a7da462 Compare January 15, 2026 16:07
@oech3 oech3 changed the title coreutils: Fail with invalid binary name tests/misc/coreutils.sh: Fail with invalid binary name Jan 15, 2026
@oech3 oech3 force-pushed the core-1 branch 3 times, most recently from 441d2a1 to ff88392 Compare January 16, 2026 06:03
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.

coreutils: Unknown program should fail (tests/misc/coreutils.sh)

1 participant