-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 doesn't work inside folders with some characters in the name #12902
Comments
Yep, I can confirm that. The good news is that it seems to have already been fixed in the main branch, so if you can build Nu from source, that should resolve this issue. |
TLDR: I'm not a Rust developer and can't help you with that.With all due respect, I'm frustrated. I downloaded this package, tried the first command and folder, and faced this error. I spent three hours trying to understand why I was so stupid, and it didn't work. It's one of the crucial commands that can be in a shell. It just doesn't fit in my mind How incoming input can affect the call of the function and parameters. It looks like a leaking of execution control to input data, which is a security issue. I would suggest reconsidering some architectural decisions. |
Looks like a duplicate of #12429 which was closed with #12625. Reading through the root issue, I don't see any security concerns there. Sorry to hear you ran into this on your try. Globbing, path handling, strings->path, etc. are areas that have seen a lot of recent changes, and I'm expecting (as a user) that it's still not finalized. |
I won't insist on anything because I don't know. I like the idea behind this project. However, from a high-level review, it's less popular than the fish shell, so the base of people who would report issues should be smaller. Still, there are three times more issues. Maybe you could prioritize the stabilization instead of the new features. |
I've used Fish for many years, and it's a great shell. While I've recently switched to Nushell as my daily-driver, I can certainly understand that you want to use a more stable shell. For me, the advantages of Nushell now outweigh its disadvantages, but that will differ for each individual's needs. However, I do think that "total issues" is a poor comparison between the two shells, for multiple reasons. I don't think anyone would disagree that there are more issues with Nu at the moment than with Fish. That's pretty much expected when:
And Nushell is, arguably, a far more ambitious project:
Finally, I'm not sure how to truly gauge "popularity" of the two shells. It's interesting to note that Nushell has 30k stars on GitHub compared to Fish's 25k, but that could simply be due to Nushell being in active development, so more people gravitate to the GitHub site than with Fish. Side-note: While GitHub has me listed as a "contributor", I'm not a Rust developer either. I've just fixed a few help topics, docs, and scripts. I'm pretty much just an end-user like you. |
As per the discussion, the issue can be closed. |
Describe the bug
ls doesn't work inside folders with some characters in the name
How to reproduce
Expected behavior
list file
Screenshots
Configuration
~/Downloads/test/ab[c> version | transpose key value | to md --pretty 05/18/2024 03:11:55 AM
Additional context
Text version of screenshot
The text was updated successfully, but these errors were encountered: