-
Notifications
You must be signed in to change notification settings - Fork 37
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
Too many levels of symbolic links #317
Comments
Thanks for reporting this @adrianlzt! I understand the problem. Not sure what's the best way to solve it though. Need to think for a bit. (some vague concerns around uniqueness of paths to documents and also resolving symlinks to folders outside of the root folder; maybe reading workspace folder in a BFS fashion and picking the shortest paths could be enough...) |
Maybe just ignoring symlinks? Or an option to ignore them. |
Also hitting this on the fwupd repo. I didn't look at the impl too closely but the Rust walkdir crate has some logic to do cycle detection by tracking the parent dirs. Maybe that's useful 🙂 https://github.com/BurntSushi/walkdir/blob/2.5.0/src/lib.rs#L973 Thanks for this nice LSP! :) |
Thanks for the kind words @tmuehlbacher!
This https://github.com/fwupd/fwupd repo? Yeah, this needs to be looked at. But it's been a pretty niche use-case so far, so didn't make it to the top of the list. |
Yes, that's the repo. For this particular case, I have a local workaround by extending the gitignore: $ echo src/tests/sys >>.gitignore |
I have a directory with many markdown files used to take notes. In that directory structure there are symbolic links doing circular paths. For example:
/home/adrian/adrianRepo/web/apps/google/programacion -> ../../../programacion/google
/home/adrian/adrianRepo/programacion/google/apps -> /home/adrian/adrianRepo/web/apps/google
This seems to configure marksman returning this error:
I'm not sure how this could be fixed, but I think circular symbolic links could be normal.
Thanks!
The text was updated successfully, but these errors were encountered: