Skip to content

fix: support ls -n on Windows NFS#12640

Open
a-rookie-of-C-language wants to merge 1 commit into
uutils:mainfrom
a-rookie-of-C-language:fix/windows-nfs-ls-n
Open

fix: support ls -n on Windows NFS#12640
a-rookie-of-C-language wants to merge 1 commit into
uutils:mainfrom
a-rookie-of-C-language:fix/windows-nfs-ls-n

Conversation

@a-rookie-of-C-language
Copy link
Copy Markdown

Summary

  • Fix ls -n on Windows NFS filesystems by reading the NfsV3Attributes EA exposed by Services for UNIX compatible NFS drivers.
  • Use the NFS uid/gid values for Windows numeric owner/group display.
  • Cache numeric uid/gid per ls entry to avoid repeated EA queries.

Validation

  • cargo check --manifest-path deps\coreutils\Cargo.toml -p uu_ls
  • cargo run --manifest-path deps\coreutils\Cargo.toml -p uu_ls -- -n src\main.rs

@cakebaker
Copy link
Copy Markdown
Contributor

@a-rookie-of-C-language your PR contains many unrelated commits.

@a-rookie-of-C-language a-rookie-of-C-language force-pushed the fix/windows-nfs-ls-n branch 4 times, most recently from 7c8f0cf to 312e1f7 Compare June 5, 2026 15:44
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

GNU testsuite comparison:

Skip an intermittent issue tests/cut/bounded-memory (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/cp/link-heap is now passing!
Congrats! The gnu test tests/tail/pipe-f is now passing!

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.

2 participants