mirror of
https://github.com/SabreTools/MPF.git
synced 2026-02-04 05:35:52 +00:00
[Problem] Check + Redumper = Error #498
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @mnadareski on GitHub (Feb 23, 2023).
Originally assigned to: @mnadareski on GitHub.
Using MPF.Check with Redumper currently does the absolutely wrong thing. It doesn't parse anything and it says the dumping program was
ddas a result. Fix this please thanks.@mnadareski commented on GitHub (Feb 24, 2023):
TL;DR on investigation:
ddto be selected from everything I can tell is for you to pass in "dd" as the dumping program. The default dumping program is DIC.-u) and an all-lower-case "redumper". Hiccup's original one used-uand sentence case "Redumper". acrid's used--useand sentence case "Redumper"ToLowerInvariantin the code to handle exactly thatSo, somehow,
--use Redumperis being interpreted as--use dd@acridAxid commented on GitHub (Feb 24, 2023):
I'm using the current stable build from GitHub.
mpf.check cdrom audio --use Redumper <my-image>.cueHere's the full output, which looks like it made no attempt to parse the image at all:
!submissionInfo.txt
@acridAxid commented on GitHub (Feb 24, 2023):
Output from the now-current build from AppVeyor:
!submissionInfo.txt
The stable build creates a
.zipfile named after the image file; when--use Aaruis passed, this.zipfile contains the!submissionInfo.txtas well as all the Aaru-generated logging, and when--use Redumperis passed, the.zipfile contains nothing but the bungled!submissionInfo.txtfrom above.The current build only creates the
!submissionInfo.txtand does not generate the.zipfile. Not sure if that's an intended artifact of the testing build.@mnadareski commented on GitHub (Feb 25, 2023):
The zip was enabled by default erroneously with last stable. The lack of zip is intentional unless you use
-z/--zip. That is incidental to the issue at hand.@mnadareski commented on GitHub (Feb 25, 2023):
I have a feeling that those who were having issues all were using a stable build. Redumper support was not completed as of the last stable and may have included placeholder info. This would explain why
ddis showing up for those users. I'm closing this because it's not an actual issue since the problem has been fixed in WIP builds.