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

is_dir: do not check for existence of directory #811

Merged
merged 2 commits into from
Aug 8, 2021
Merged

is_dir: do not check for existence of directory #811

merged 2 commits into from
Aug 8, 2021

Conversation

sharkdp
Copy link
Owner

@sharkdp sharkdp commented Aug 8, 2021

This fixes a bug on Windows where fd could not be used on ram disks
and encrypted folders.

closes #752

@sharkdp
Copy link
Owner Author

sharkdp commented Aug 8, 2021

build from this branch for testing: fd-v8.2.1-x86_64-pc-windows-msvc.zip.zip

@sharkdp
Copy link
Owner Author

sharkdp commented Aug 8, 2021

The test with a ImDisk virtual drive was successful.

But now the actual

[fd error]: Could not retrieve current directory (has it been deleted?).

check doesn't work anymore and fd just exits silently.

This fixes a bug on Windows where `fd` could not be used on ram disks
and encrypted folders.

closes #752
@sharkdp
Copy link
Owner Author

sharkdp commented Aug 8, 2021

The new approach should hopefully fix both issues. And maybe even #365

@sharkdp sharkdp force-pushed the fix-752 branch 3 times, most recently from a6e205a to aa91121 Compare August 8, 2021 12:43
@sharkdp
Copy link
Owner Author

sharkdp commented Aug 8, 2021

@sharkdp sharkdp merged commit aeff525 into master Aug 8, 2021
@sharkdp sharkdp deleted the fix-752 branch August 8, 2021 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

fd cannot search files under a RAM disk
1 participant