-
Notifications
You must be signed in to change notification settings - Fork 20
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
Package feature doesn't work in the GUI on Windows: Protocol not found #918
Comments
Thanks for this. I believe the mktemp error you're mentioning was fixed in 1be1d83. Could you retry with the latest daily build at https://mediaarea.net/download/snapshots/binary/dvrescue/? But I suspect the "protocol not found" message is indicating that the ffmpeg build bundled into the dvrescue app for windows is missing the subfile protocol which dvpackager depends on. @JeromeMartinez, could you confirm that the bundled windows ffmpeg build includes the subfile input protocol (it should be listed in |
@dericed The daily snapshots don't seem to include the full Windows build with GUI for some reason. Since these are text shell scripts, I have instead manually edited the shell scripts from the portable version of 24.07 using the changes at 1be1d83 This appears to have fixed the mktemp error. However, the conversion is still failing. Here's the full debug log when I reran it:
I found a file
If I try manually running that command with the bundled FFmpeg included with Windows GUI, I get the same protocol error:
Here is the result of running `ffmpeg -protocols`:
I have a copy of FFmpeg 7.0.1, so I tried temporarily swapping that in. This time, I got a more verbose error:
Is it trying to create files with |
@dericed ah! I think the problem is that this part of the ffmpeg command line was not properly quoted:
If I try running with I do see that there is already some quoting at Line 1226 in b835441
so somehow it is being subsequently lost somewhere... |
@dericed I think this is the problem:
If I change it to:
then it works, and I get the expected
|
Thanks, I just merged #920, so @JohnstonJ please test the next windows daily build at https://old.mediaarea.net/download/snapshots/binary/dvrescue-gui/ (whatever comes after 0729) |
@dericed I retested with the snapshot from https://mediaarea.net/download/snapshots/binary/dvrescue-gui/20240731/ .... I'm not encountering any of the errors mentioned in this issue any more, so I'll go ahead and close the issue.... The remaining problem is tracked in #927. Thanks for the quick fixes! |
The file will then fail to convert in the queue. Here are the debug logs. Note the error from mktemp, and subsequent errors where it's unable to write files:
The text was updated successfully, but these errors were encountered: