Skip to content

cut: strip i/o errno - #14060

Open
Devel08 wants to merge 4 commits into
uutils:mainfrom
Devel08:fix-issue-14051
Open

cut: strip i/o errno#14060
Devel08 wants to merge 4 commits into
uutils:mainfrom
Devel08:fix-issue-14051

Conversation

@Devel08

@Devel08 Devel08 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

closes #14051

run on linux
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/date/date-locale-hour (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/retry (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/cut/cut-huge-range is now passing!
Congrats! The gnu test tests/tail/tail-n0f is now passing!
Skip an intermittent issue tests/pr/bounded-memory (was skipped on 'main', now failing)

@Devel08

Devel08 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

really interesting why cargo tests cannot reproduce that

$ cargo run cut -c1 /proc/self/mem
    Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.13s
     Running `target/debug/coreutils cut -c1 /proc/self/mem`
cut: Input/output error

@codspeed-hq

codspeed-hq Bot commented Aug 21, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 357 untouched benchmarks
⏩ 50 skipped benchmarks1


Comparing Devel08:fix-issue-14051 (57d4288) with main (df30282)

Open in CodSpeed

Footnotes

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

@Devel08

Devel08 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

@cakebaker what cargo test returns is different than when the same command it uses is ran manually, what can be the cause?

@Devel08

Devel08 commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

turns out we had to strip errno for both chars and bytes :D

@Devel08
Devel08 marked this pull request as ready for review August 21, 2026 13:28
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.

cut: unstripped i/o error

1 participant