Skip to content

fix: support ls -n on Windows NFS#87

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

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

Conversation

@a-rookie-of-C-language

Copy link
Copy Markdown

close #86

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

Note: I verified the non-NFS fallback path locally; I do not have an NFS mount available to runtime-test the EA path.

@lhecker

lhecker commented Jun 5, 2026

Copy link
Copy Markdown
Member

This is a git submodule. You have to push the submodule commit into a fork of microsoft/uutils-coreutils for me to see it.

@a-rookie-of-C-language a-rookie-of-C-language force-pushed the fix/windows-nfs-ls-nfs branch 6 times, most recently from a03565a to a7b4133 Compare June 5, 2026 15:44
@lhecker lhecker added the C-bug It shouldn't be doing this. label Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-bug It shouldn't be doing this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ls.exe -n does not work on NFS filesystems

2 participants