-
Notifications
You must be signed in to change notification settings - Fork 31
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
Remaining temp files #279
Comments
I am also experiencing this after the newest version [0.9.1] with the sample-encode command as well. |
It's possible a recent ffmpeg process is still alive and prevents cleanup. I recently removed logic that would, a little too eagerly, kill ffmpeg so this makes sense. So I'll look at adding graceful shutdown of these child processes which should resolve this. |
I've implemented awaiting for child processes, like ffmpeg, to exit before temp file cleanup. Hopefully this should resolve this issue, though I couldn't actually reproduce it myself. If possible can you let me know if this resolves the issue, you can install the latest code via cargo. |
Maybe so. Windows Explorer starts generating thumbnails from videos and this means that a process hangs on the files that prevents them from being deleted. I just don't know how long the timeout is. |
crf-search does not delete all temp files after finishing the search.
The both files of the last sample are staying in the temp folder. The temp folder won't be removed.
(Windows version but I guess that this does not matter)
The text was updated successfully, but these errors were encountered: