Skip to content

tests(readlink): enable wasi integration tests - #15322

Merged
cakebaker merged 1 commit into
uutils:mainfrom
eduardomourar:feat/wasi-readlink-tests
Oct 20, 2026
Merged

cakebaker merged 1 commit into
uutils:mainfrom
eduardomourar:feat/wasi-readlink-tests

Conversation

@eduardomourar

Copy link
Copy Markdown
Contributor

Enable test_readlink:: for wasm32-wasip1 in CI, ignoring cases incompatible with the WASI sandbox: canonicalization resolves against the guest's virtual root rather than the host's absolute path, and reading a symlink whose stored target is itself an absolute host path fails even without following it.

Comment thread tests/by-util/test_readlink.rs Outdated
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

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

@sylvestre

Copy link
Copy Markdown
Contributor

sorry but it needs to be rebased

@eduardomourar
eduardomourar force-pushed the feat/wasi-readlink-tests branch 2 times, most recently from e5cdfa2 to 3d21f17 Compare October 18, 2026 23:04
@eduardomourar
eduardomourar force-pushed the feat/wasi-readlink-tests branch from 3d21f17 to c9a8d19 Compare October 19, 2026 08:44
@eduardomourar
eduardomourar force-pushed the feat/wasi-readlink-tests branch from c9a8d19 to c54ade1 Compare October 19, 2026 09:00
@cakebaker
cakebaker merged commit 3d7d7a5 into uutils:main Oct 20, 2026
89 checks passed
@cakebaker

Copy link
Copy Markdown
Contributor

Thanks!

@eduardomourar
eduardomourar deleted the feat/wasi-readlink-tests branch October 20, 2026 22:42
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