Skip to content

tail: report PermissionDenied instead of No such file when metadata f…#11018

Merged
cakebaker merged 9 commits intouutils:mainfrom
RedNhight:fix-issue-10193
Feb 23, 2026
Merged

tail: report PermissionDenied instead of No such file when metadata f…#11018
cakebaker merged 9 commits intouutils:mainfrom
RedNhight:fix-issue-10193

Conversation

@RedNhight
Copy link
Copy Markdown
Contributor

Fixes #10193

Before:
tail prints "No such file" on lack of permissions.

After:
tail attempts open() and prints correct Permission denied.

@oech3
Copy link
Copy Markdown
Contributor

oech3 commented Feb 18, 2026

Thankyou for 1st contrib. Would you add test to https://github.com/uutils/coreutils/blob/main/tests/by-util/test_tail.rs ?

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Congrats! The gnu test tests/date/date-locale-hour is no longer failing!

@RedNhight
Copy link
Copy Markdown
Contributor Author

Thankyou for 1st contrib. Would you add test to https://github.com/uutils/coreutils/blob/main/tests/by-util/test_tail.rs ?

Yes, sure!

@rglKali
Copy link
Copy Markdown

rglKali commented Feb 18, 2026

lgtm!

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

GNU test failed: tests/misc/io-errors. tests/misc/io-errors is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/pr/bounded-memory (fails in this run but passes in the 'main' branch)

@cakebaker
Copy link
Copy Markdown
Contributor

Can you please run cargo fmt and cargo clippy? Thanks.

@RedNhight
Copy link
Copy Markdown
Contributor Author

Can you please run cargo fmt and cargo clippy? Thanks.

Yes, sure!

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/tail/symlink (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/cut/bounded-memory is no longer failing!

@RedNhight
Copy link
Copy Markdown
Contributor Author

Can you please run cargo fmt and cargo clippy? Thanks.

Done!

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

GNU test failed: tests/tail/retry. tests/tail/retry is passing on 'main'. Maybe you have to rebase?

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

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

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

GNU test failed: tests/date/date-locale-hour. tests/date/date-locale-hour is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/pr/bounded-memory (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/tail/tail-n0f is now passing!

RedNhight and others added 3 commits February 22, 2026 22:29
Replace make_file with touch

Co-authored-by: Daniel Hofstetter <daniel.hofstetter@42dh.com>
Simplify tests

Co-authored-by: Daniel Hofstetter <daniel.hofstetter@42dh.com>
@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Congrats! The gnu test tests/date/date-locale-hour is no longer failing!

@cakebaker cakebaker merged commit e330d2c into uutils:main Feb 23, 2026
157 checks passed
@cakebaker
Copy link
Copy Markdown
Contributor

Thanks for your PR!

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.

tail: exists precheck masks permission errors

4 participants