-
Notifications
You must be signed in to change notification settings - Fork 30
ls implementation #71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
One comment, in re args parsing and long options: For the common case where an
|
I'm pretty sure it's already doing that with the exception of |
Does not build on MacOS,
(continues for many pages) |
This passes all of coreutils ls' tests excluding non-POSIX ones.
Two tests have quirks though:
test_ls_file_type
skips block/character devices if not run withsudo -E cargo ...
. Those need elevated permission to create.test_ls_time
will stall a bit because it needs to wait for the ctime of the file to change. This is already an improvement over the 4 second wait time of the original.There's also the
-k
option which coreutils always has enabled. Specification says to use 512-byte blocks if-k
isn't specified but they always have it as 1024-byte blocks with or without-k
.