]> git.ipfire.org Git - thirdparty/rsync.git/commit
testsuite/xattrs: ignore SUNWattr_* in the Solaris xls helper
authorAndrew Tridgell <andrew@tridgell.net>
Wed, 29 Apr 2026 22:18:01 +0000 (08:18 +1000)
committerAndrew Tridgell <andrew@tridgell.net>
Wed, 29 Apr 2026 23:25:58 +0000 (09:25 +1000)
commitdcf364dac5970ffdd5279387c8a3fbe8d6cb8071
treee4e36e98849a780f2d13574741f34c01bd846297
parentd1eff8f0dca141c226512ad925875e8403bce6be
testsuite/xattrs: ignore SUNWattr_* in the Solaris xls helper

The Solaris xls() function listed every entry in the file's xattr
directory, which on Solaris includes OS-managed SUNWattr_ro and
SUNWattr_rw pseudo-attributes. SUNWattr_rw embeds the file creation
time, so its bytes naturally differ between the source and destination
files, making the xattrs and xattrs-hlink tests fail with diffs that
have nothing to do with rsync.

Rsync's own listxattr wrapper already filters these out
(lib/sysxattrs.c), so the right fix is to filter them in the test
display too. Other platforms are unaffected because each has its own
xls() branch in the case statement.

With the test now actually passing on Solaris, drop the CI hack that
overwrote testsuite/xattrs.test with a skip stub.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
.github/workflows/solaris-build.yml
testsuite/xattrs.test