Skip Menu |
 

This queue is for tickets about the CDDB CPAN distribution.

Report information
The Basics
Id: 131
Status: resolved
Worked: 15 min
Priority: 0/
Queue: CDDB

People
Owner: Nobody in particular
Requestors: hardy42 [...] gmx.net
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 1.07
Fixed in: (no value)

Attachments


Subject: Patch: Implement freedb protocol levels
Download (untitled) / with headers
text/plain 438b
*Rocco: sorry for the duplicate, but found the 'official' way now...* In the CDDB-1.07 module I missed the ability to change the freedb "CDDB Protocol Level" (so I can see the additional info defined in the newer protocol levels). The attached patch adds this functionallity to the module. I have tested a little with several CD's (and servers) and couldn't find any problems so far, so hopefully it will not break anything ;-) hardy
Download CDDB.pm-diff
application/octet-stream 1.8k

Message body not shown because it is not plain text.

Download (untitled) / with headers
text/plain 473b
Someone else supplied a similar patch for this, but it didn't include a way to override the protocol level. I've applied your patch with some minor changes. The protocol parameter, for instance, is named "Protocol_Version" instead of "CDDB_Protocol". I also documented the new parameter. Hopefully I can resolve other open issues and release version 1.08 this weekend. Thanks for using the request tracker. -- Rocco Caputo / troc@pobox.com / poe.perl.org / poe.sf.net


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.