Skip to content

Conversation

@sudhackar
Copy link
Contributor

$ strace -e llistxattr,listxattr,getxattr ./target/debug/coreutils ls -lLd l
listxattr("l", 0x1, 0)                  = 24
listxattr("l", "system.posix_acl_access\0", 24) = 24
drwxrwxr-x+ 2 sudhackar sudhackar 4096 Jun 18 16:47 l
+++ exited with 0 +++

as compared to no +

Signed-off-by: Sudhakar Verma <[email protected]>
@sudhackar
Copy link
Contributor Author

This should fix #8216

Signed-off-by: Sudhakar Verma <[email protected]>
Signed-off-by: Sudhakar Verma <[email protected]>
@cakebaker cakebaker linked an issue Jun 18, 2025 that may be closed by this pull request
@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Note: The gnu test tests/ls/no-cap is now being skipped but was previously passing.

@sudhackar
Copy link
Contributor Author

I have fixed the patch for the GNU test - that passes now instead of SKIP

Signed-off-by: Sudhakar Verma <[email protected]>
@github-actions
Copy link

GNU testsuite comparison:

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

@sudhackar
Copy link
Contributor Author

tests/misc/tee seems flaky locally - failed 1 out of 10 runs for me

@sylvestre
Copy link
Contributor

we know, just ignore it :)

@sylvestre
Copy link
Contributor

(but don't hesitate to have a look to fix it if you can reproduce it easily :)

@sylvestre sylvestre merged commit c674cf1 into uutils:main Jun 19, 2025
75 checks passed
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.

ls: doesn't list acl "+" status for symlinks

3 participants