-
Notifications
You must be signed in to change notification settings - Fork 2k
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
CHIA-1697: Add new flag to support recursively scanning and following links #18803
Conversation
It seems the hardlink test was skipped on all the OSs. A little more research tells me hard links don't work for directories at all on any of the main OS (macos, linux, windows) |
Pull Request Test Coverage Report for Build 11632202557Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks good. However, the bug report you reference is talking about hard links, not symbolic links. I suppose it may be a bit tricker to test as not all filesystems may support them. But did you test this manually with hard links too?
I see the term hard-links there. But, is that much of a thing? I thought hard links didn't even show as being a link since they are just a regular whatever like the first whatever. They also presumably couldn't exist across volumes like are shown there? I know I'm extrapolating a bit here.
Do you know of a filesystem that supports directory hard links? |
oh yeah, those do look like soft links |
Yea, I think the reporter was just incorrect but I didn't want to correct the report. My research indicates hard links to directories aren't something that any OS supports at least in the normal filesystems. |
Fixes #16268
Adds new harvester option
follow_links
that only works when used withrecursive_scan
and usesglob.glob
instead ofPath.rglob
in order to follow links when scanning recursively for plots.