Skip to content

cp, tail: omit pipe() if input or output is pipe#12359

Open
oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:cpstream-omit-pipe
Open

cp, tail: omit pipe() if input or output is pipe#12359
oech3 wants to merge 1 commit into
uutils:mainfrom
oech3:cpstream-omit-pipe

Conversation

@oech3
Copy link
Copy Markdown
Contributor

@oech3 oech3 commented May 18, 2026

We don't need additional 1 MiB pipe for stream input. Closes #12358 .

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 18, 2026

GNU testsuite comparison:

Skipping an intermittent issue tests/date/date-locale-hour (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)

@oech3 oech3 marked this pull request as ready for review May 18, 2026 07:08
@oech3 oech3 force-pushed the cpstream-omit-pipe branch from 44a644f to a1102dd Compare May 18, 2026 08:45
Comment thread src/uucore/src/lib/features/buf_copy/linux.rs
@oech3 oech3 force-pushed the cpstream-omit-pipe branch 2 times, most recently from 3cd902f to 3f8991e Compare May 18, 2026 15:32
@oech3
Copy link
Copy Markdown
Contributor Author

oech3 commented May 18, 2026

tail has same issue.

@oech3 oech3 force-pushed the cpstream-omit-pipe branch from 3f8991e to 7e64abe Compare May 18, 2026 17:31
@oech3 oech3 changed the title cp: omit pipe() if input is pipe cp, tail: omit pipe() if input is pipe May 18, 2026
@oech3 oech3 changed the title cp, tail: omit pipe() if input is pipe cp, tail: omit pipe() if input or output is pipe May 18, 2026
Comment thread src/uucore/src/lib/features/buf_copy/linux.rs
@oech3
Copy link
Copy Markdown
Contributor Author

oech3 commented May 18, 2026

There is doc issue and unnecessary wrap on splice_unbounded. But I wait merging this.

@xtqqczze
Copy link
Copy Markdown
Contributor

The use of std::io::Result<bool>, where true represents a fallback state rather than success, is confusing. It would be clearer to refactor this first: #12365.

@oech3
Copy link
Copy Markdown
Contributor Author

oech3 commented May 19, 2026 via email

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.

cp, tail: unnecessary pipe call

2 participants