Skip to content

cp/mktemp: handle closed stdout errors for GNU compatibility#10494

Draft
ChrisDryden wants to merge 5 commits into
uutils:mainfrom
ChrisDryden:fix-cp-stdout-write-errors
Draft

cp/mktemp: handle closed stdout errors for GNU compatibility#10494
ChrisDryden wants to merge 5 commits into
uutils:mainfrom
ChrisDryden:fix-cp-stdout-write-errors

Conversation

@ChrisDryden

Copy link
Copy Markdown
Collaborator

This is an in progress PR that I am publishing to give an example in the comments in the other PR's how we can use the signal handling changes in the init_data section to handle the stdout error handling the same way the GNU utilities handle it. Just need to fix the windows issues and add more integration testing.

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/tail/inotify-dir-recreate (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/misc/close-stdout is no longer failing!

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

GNU test failed: tests/date/date-ethiopia. tests/date/date-ethiopia is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/date/date-iran. tests/date/date-iran is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/date/date-thailand. tests/date/date-thailand is passing on 'main'. Maybe you have to rebase?
Congrats! The gnu test tests/misc/close-stdout is no longer failing!

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Congrats! The gnu test tests/misc/close-stdout is no longer failing!

@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Congrats! The gnu test tests/pr/bounded-memory is no longer failing!

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.

2 participants