Skip to content

yes: avoid ambiguous PAGE_SIZE#12370

Merged
sylvestre merged 1 commit into
uutils:mainfrom
oech3:yes-atomic
May 20, 2026
Merged

yes: avoid ambiguous PAGE_SIZE#12370
sylvestre merged 1 commit into
uutils:mainfrom
oech3:yes-atomic

Conversation

@oech3
Copy link
Copy Markdown
Contributor

@oech3 oech3 commented May 19, 2026

This is actually same with page size. But this is used to checking that partial tee is allowed. So explicitly use The maximum length at which writes to a pipe are atomic.

@oech3 oech3 marked this pull request as ready for review May 19, 2026 02:00
@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

GNU test failed: tests/timeout/timeout-group. tests/timeout/timeout-group is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/misc/io-errors (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/cut/bounded-memory (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/tail/tail-n0f (passes in this run but fails in the 'main' branch)

@xtqqczze
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

@sylvestre sylvestre left a comment

Choose a reason for hiding this comment

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

LGTM — PIPE_BUF is semantically correct here (atomic write guarantee).

@sylvestre sylvestre merged commit 8a271f5 into uutils:main May 20, 2026
172 of 173 checks passed
@oech3 oech3 deleted the yes-atomic branch May 20, 2026 16:50
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.

3 participants