Skip to content

Conversation

@BenWiederhake
Copy link
Collaborator

This is tested for in the GNU shred-passes test, so we don't have a choice if we want to stay compatible.

Found while getting ready for #5711.

This is tested for in the GNU shred-passes test, so we don't have a
choice if we want to stay compatible.
@BenWiederhake
Copy link
Collaborator Author

CI failure due to flaky test_tail::test_retry9. See also #7756: These tests are flaky by nature.

@github-actions
Copy link

GNU testsuite comparison:

GNU test failed: tests/misc/tee. tests/misc/tee is passing on 'main'. Maybe you have to rebase?
Skipping an intermittent issue tests/timeout/timeout (passes in this run but fails in the 'main' branch)

@BenWiederhake
Copy link
Collaborator Author

I don't understand why misc/tee failed in this CI run, but it doesn't seem plausible that a change in shred caused it. The test log only says:

FAIL: tests/misc/tee
====================

tee: file.ro: Permission denied (os error 13)
FAIL tests/misc/tee.sh (exit status: 1)

So that's not really helpful.

Passes on my machine with this branch 😕

PASS: tests/misc/tee.sh
============================================================================
Testsuite summary for GNU coreutils 9.7-modified
============================================================================
# TOTAL: 1
# PASS:  1
# SKIP:  0
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
============================================================================

@jtracey
Copy link
Contributor

jtracey commented Apr 20, 2025

tee failure is a false positive, see the above issue mention

@cakebaker cakebaker merged commit 06bae0d into uutils:main Apr 21, 2025
68 of 70 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.

3 participants