Skip to content

readlink: Set silent mode as default#10711

Open
denendaden wants to merge 2 commits intouutils:mainfrom
denendaden:iss10578
Open

readlink: Set silent mode as default#10711
denendaden wants to merge 2 commits intouutils:mainfrom
denendaden:iss10578

Conversation

@denendaden
Copy link

This primarily changes readlink to be silent rather than verbose by default, in accordance with GNU readlink. It also makes a couple more minor changes, to bring behavior in line with GNU:

  • -s, -v, and -q can now be duplicated, and the last one takes priority
  • when POSIXLY_CORRECT is set, -s and -q are ignored entirely and it is always verbose (I am not sure if this is intentional behavior on GNU's part but it is how it currently seems to work)

Fixes #10578

@sylvestre
Copy link
Contributor

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

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

GNU testsuite comparison:

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

@ChrisDryden
Copy link
Collaborator

Seems like the windows tests are failing because its expecting the platform specific error message:

.stderr_contains("Invalid argument")

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.

readlink: Behaviour differs from GNU Coreutils when path exists but is not a link

3 participants