Skip to content

Adding temporary root owned file in tmp folder for eperm test#9525

Closed
ChrisDryden wants to merge 1 commit intouutils:mainfrom
ChrisDryden:eperm
Closed

Adding temporary root owned file in tmp folder for eperm test#9525
ChrisDryden wants to merge 1 commit intouutils:mainfrom
ChrisDryden:eperm

Conversation

@ChrisDryden
Copy link
Collaborator

@ChrisDryden ChrisDryden commented Nov 29, 2025

This test is passing locally and failing on the build fleet because there's no files in the /tmp folder that are not owned by the runner of the test. Adding this to the build-gnu step to create this file so that there's something that the test can use and pass.

fail-eperm.xpl: couldn't find a file not owned by you
in any of the following directories:
/tmp /var/tmp /usr/tmp
...so, skipping this test
SKIP: tests/rm/fail-eperm.xpl

@ChrisDryden
Copy link
Collaborator Author

Looks like the test passed now 🥳 I rebased to get the change that shows when something turns from skipped to pass so should be getting that comment soon

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/rm/fail-eperm is now passing!

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/overlay-headers (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/rm/fail-eperm is now passing!

@ChrisDryden
Copy link
Collaborator Author

A more concise implementation is here: #9531

@oech3
Copy link
Contributor

oech3 commented Nov 30, 2025

Let's backport #9531 (comment) for simplicity at coreutils > 9.9 instead.

@oech3
Copy link
Contributor

oech3 commented Dec 1, 2025

This test was removed(merged to another test coreutils/coreutils@ffbd8c4) by @pixelb .

What should we do at here in the case at here?

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