pacpl doesn't work with latest Audio::APETags (patch to use Audio::Scan instead)

Bug #892610 reported by Ryan Thompson
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pacpl (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

The latest Audio::APETags perl module is just a wrapper for Audio::Scan, which should be used instead. It does not have the same interface as the old version of Audio::APETags, so pacpl fails when used with it.

I am attaching a patch that makes pacpl use Audio::Scan instead of Audio::APETags, and updates the relevant code to use the new interface.

With this patch, you must add libaudio-scan-perl to the dependencies (and maybe remove libaudio-musepack-perl, unless pacpl uses other modules from that package).

Tags: patch
Revision history for this message
Ryan Thompson (rct86) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "Use Audio::Scan instead of Audio::APETags" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Ryan Thompson (rct86) wrote :

The upstream author of pacpl has accepted this patch, and has stated that he will include it in the next release.

Revision history for this message
Dominik Stadler (dominik-stadler) wrote :

There is now a ppa-package available with this change for Lucid at https://launchpad.net/~dominik-stadler/+archive/ppa/

Revision history for this message
Ryan Thompson (rct86) wrote :

pacpl as of 12.04 (possibly earlier) includes the fix proposed here, so I'm marking this as Fix Released.

Changed in pacpl (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.