Skip to content

Conversation

tgrez
Copy link
Contributor

@tgrez tgrez commented Jul 2, 2025

Fixes #8215

@sylvestre
Copy link
Contributor

could you please add a test to make sure we don't regress in the future? thanks

Copy link

github-actions bot commented Jul 2, 2025

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/stdbuf (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/timeout/timeout (passes in this run but fails in the 'main' branch)

@tgrez
Copy link
Contributor Author

tgrez commented Jul 3, 2025

A test checking regression added and verified that it fails without the fix (should have started with the test;)

However, AFAICT MacOS and Windows filepaths have length limits (1024 and 260 respectively) and the tests still add a /tmp directory, so I limited the test to run only for Linux. Hope it's okay.

Copy link

github-actions bot commented Jul 3, 2025

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/tail/inotify-dir-recreate (passes in this run but fails in the 'main' branch)
Skipping an intermittent issue tests/timeout/timeout (passes in this run but fails in the 'main' branch)

@cakebaker cakebaker merged commit 6584c99 into uutils:main Jul 4, 2025
76 checks passed
@cakebaker
Copy link
Contributor

Thanks!

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.

rm: "Not a directory" error
3 participants