mirror of
https://github.com/SabreTools/MPF.git
synced 2026-02-04 05:35:52 +00:00
[Problem] Only in redumper, BD-ROM-66 is reported incorrectly #663
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 @fuzz6001 on GitHub (Jan 2, 2024).
Originally assigned to: @mnadareski on GitHub.
Version
What version are you using?
Build
What runtime version are you using?
Describe the issue
Only in redumper, BD-ROM-66 is reported incorrectly.
In DIC, BD-ROM-66 is reported.
Incidentally, the PIC files for both match.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Output media type BD-ROM-66.
Additional context
redumper !submissionInfo.txt
redumper logs
DIC !submissionInfo.txt
@Deterous commented on GitHub (Jan 3, 2024):
@fuzz6001 would you be able to test this build for me with the same disc/dump: https://ci.appveyor.com/project/Deterous/mpf/build/artifacts
I have attempted to fix it with this change:
5521f87e29@fuzz6001 commented on GitHub (Jan 3, 2024):
@Deterous
Just now I finished fixing it and was creating PR.That code works. Thanks for fixing!
@Deterous commented on GitHub (Jan 3, 2024):
Perfect. If you had solved it differently, let me know, otherwise I'll submit this change as a PR.
@fuzz6001 commented on GitHub (Jan 3, 2024):
I copied the code for DIC.
3125ec997532c12e1332/MPF.Core/Modules/DiscImageCreator/Parameters.cs (L469-L484)@Deterous commented on GitHub (Jan 3, 2024):
Alright, that's the same as what I did, but only the two lines about GetPICIdentifier are needed as the redumper log already provides the layerbreak values (I think).
@mnadareski commented on GitHub (Jan 3, 2024):
Fixed by
8cb0b37e80