Skip to content
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

ls: --l should output the same as --literal #5025

Closed
cakebaker opened this issue Jun 30, 2023 · 3 comments · Fixed by #5055
Closed

ls: --l should output the same as --literal #5025

cakebaker opened this issue Jun 30, 2023 · 3 comments · Fixed by #5055
Labels

Comments

@cakebaker
Copy link
Contributor

GNU ls uses ls --l as a shortcut for --literal:

$ ls --l
aa.txt  foo.8b79  foo.eWmR  hello.txt  test_dir

$ ls --literal
aa.txt  foo.8b79  foo.eWmR  hello.txt  test_dir

uutils ls, on the other hand, uses ls --l as a shortcut for --long (our extension to GNU ls):

$ cargo run ls --l
total 12
-rw-r--r-- 1 dho dho    2 Apr 10 16:04 aa.txt
-rw------- 1 dho dho    0 May 16 09:31 foo.8b79
-rw------- 1 dho dho    0 May 16 09:22 foo.eWmR
-rw-r--r-- 1 dho dho   12 Jun 21 14:51 hello.txt
drwxr-xr-x 2 dho dho 4096 Jun 21 11:54 test_dir

$ cargo run ls --literal
aa.txt  foo.8b79  foo.eWmR  hello.txt  test_dir
@tertsdiepraam
Copy link
Member

@cakebaker, what is your preferred solution here? Should we remove --long?

@cakebaker
Copy link
Contributor Author

I don't have a preferred solution here and the proposed solution in the PR looks good to me.

I think we should make the decision whether to remove --long independent of this issue. And because it is tested and documented, I tend to keep it.

@tertsdiepraam
Copy link
Member

I was thinking it's a bit confusing as it is now with the workaround. But I do like having --long too, so I guess we'll keep it for now.

@cakebaker cakebaker linked a pull request Jul 9, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants