Skip to content

tr: refactor test_stdin_is_socket#12276

Draft
xtqqczze wants to merge 1 commit into
uutils:mainfrom
xtqqczze:refactor-test_stdin_is_socket
Draft

tr: refactor test_stdin_is_socket#12276
xtqqczze wants to merge 1 commit into
uutils:mainfrom
xtqqczze:refactor-test_stdin_is_socket

Conversation

@xtqqczze
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 13, 2026

GNU testsuite comparison:

Skip an intermittent issue tests/cut/bounded-memory (fails in this run but passes in the 'main' branch)
Note: The gnu test tests/basenc/bounded-memory is now being skipped but was previously passing.
Note: The gnu test tests/tail/tail-n0f is now being skipped but was previously passing.
Congrats! The gnu test tests/printf/printf-surprise is now passing!

@xtqqczze
Copy link
Copy Markdown
Contributor Author

@xtqqczze xtqqczze marked this pull request as draft May 13, 2026 13:08
@xtqqczze
Copy link
Copy Markdown
Contributor Author

These two forms are not equivalent due to drop timing:

let mut x = File::from(writer);
x.write_all(b"::").unwrap();
File::from(writer).write_all(b"::").unwrap();

@xtqqczze xtqqczze force-pushed the refactor-test_stdin_is_socket branch from b1981f7 to edd123f Compare May 13, 2026 14:17
@xtqqczze
Copy link
Copy Markdown
Contributor Author

xtqqczze commented May 13, 2026

I would like to use filedescriptor::socketpair from the filedescriptor crate here, something like: xtqqczze@26a0afd

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.

1 participant