Converting files from .flac to .m4a fails in pacpl 4.0.5: Error code 256

Bug #971383 reported by Lars
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
pacpl (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Converting files from .flac to m4a using pacpl fails due to wrong (likely outdated) options being passed to ffmpeg 0.6.4-4 during conversion.

An easy fix is swapping current code in line 2064 from:
return "-title \"$tag_name{title}\" -track \"$tag_name{track}\" -author \"$tag_name{artist}\" -album \"$tag_name{album}\" -comment \"$tag_name{comment}\" -year \"$tag_name{year}\" -genre \"$tag_name{genre}\"";

to:
return "-metadata title=\"$tag_name{title}\" -metadata track=\"$tag_name{track}\" -metadata author=\"$tag_name{artist}\" -metadata album=\"$tag_name{album}\" -metadata comment=\"$tag_name{comment}\" -metadata year=\"$tag_name{year}\" -metadata genre=\"$tag_name{genre}\"";

There apparently are no longer options such as "-title" anymore and those options have been replaced by "-metadata key=value".
I attached a patch fixing the bug for me. The patch uses two local files pacpl.pl and pacpl.pl.new. Adjust those names to your needs before applying it.

Revision history for this message
Lars (larmeh) wrote :
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "This patch fixes it for me. The patch uses two files pacpl.pl and pacpl.pl.new. Adjust those names to your needs." 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-reviewers team 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
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in pacpl (Ubuntu):
status: New → Confirmed
tags: added: precise
Revision history for this message
Matteo Cypriani (mcy) wrote :

Can't reproduce this with 5.0.1.

Also the previous comment is spam and should be removed.

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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