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

[CUERipper] Add setting to force ReadCDCommand #319

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

c72578
Copy link
Collaborator

@c72578 c72578 commented Mar 24, 2024

The ReadCDCommand is not detected correctly in some cases. This has
been reported for Plextor drives PX-5224TA and PX-708A.

  • Add CUERipper setting ReadCDCommands
    The setting can be modified by editing CUERipper\settings.txt
  • Values for the drive specific setting ReadCDCommands are:
    0 (ReadCdBEh), 1 (ReadCdD8h), 2 (Unknown)
    Default: 2
  • Resolves:
    https://hydrogenaud.io/index.php/topic,125668.0.html

The ReadCDCommand is not detected correctly in some cases. This has
been reported for Plextor drives PX-5224TA and PX-708A.

- Add CUERipper setting `ReadCDCommands`
  The setting can be modified by editing `CUERipper\settings.txt`
- Values for the drive specific setting `ReadCDCommands` are:
  0 (ReadCdBEh), 1 (ReadCdD8h), 2 (Unknown)
  Default: 2
- Resolves:
  https://hydrogenaud.io/index.php/topic,125668.0.html
@c72578
Copy link
Collaborator Author

c72578 commented Mar 24, 2024

Here is a build for testing:

CUETools_2.2.5_2024-03-24_git_e8c795a_CUERipper_Add_setting_to_force_ReadCDCommand.zip
SHA256: 17133650b6ed528a6ac410ea9cbe943200ab8119b89b0b3dc9d2ef3188ee90ac

In order to force e.g. ReadCdD8h, edit CUERipper\settings.txt and change the default value of <int>2</int> to <int>1</int>.
Example:

 =        <string>PLEXTOR - DVDR PX-708A</string>
 =      </key>
 =      <value>
-=        <int>2</int>
+=        <int>1</int>
 =      </value>
 =    </item>
 =  </ReadCDCommands>

@ha-korth
Copy link

CUERipper should not be running when changing this setting.

@jgreene5
Copy link

Everything is working well now. Thank you so much, that was fast! I also ran a couple of non-Plextor drives simultaneously to test; all worked well.

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.

None yet

3 participants