cueprint/cuetag: no "DATE" and "GENRE" tags are read/written from/to audiofiles

Bug #896978 reported by Alexander
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
cuetools (Ubuntu)
Confirmed
Undecided
Unassigned

Bug Description

In .cue files DATE and GENRE tags are stored as "comments" (strings begin with "REM ... " ).
E.g.:
REM GENRE "Heavy Metal"
REM DATE 1998
PERFORMER "....."

And there are two problems with this:
1) cueprint doesn't treat these fields as tags (because of cue-standard?).
2) cuetag doesn't properly tag audiofiles (because of no output of these tags from cueprint)
I've made a hack in cuetag to read these two tags directly from .cue file.
Now DATE and GENRE tags are supported (only in ogg/flac files).
Maybe this breaks something. I don't know. I've only tested this on .flac files.

P.S.: looks like problem exists in Ubuntu 11.11 too.

ProblemType: Bug
DistroRelease: Ubuntu 11.04
Package: cuetools 1.3.1-9 [modified: usr/bin/cuetag]
ProcVersionSignature: Ubuntu 3.0.0-12.20-generic 3.0.4
Uname: Linux 3.0.0-12-generic x86_64
Architecture: amd64
Date: Sun Nov 27 22:36:00 2011
ProcEnviron:
 LANGUAGE=ru_UA:ru
 PATH=(custom, no user)
 LANG=ru_UA.UTF-8
 SHELL=/bin/bash
SourcePackage: cuetools
UpgradeStatus: Upgraded to natty on 2011-05-04 (207 days ago)

Revision history for this message
Alexander (lxandr) wrote :
Alexander (lxandr)
description: updated
summary: - cuetag: no "DATE" and "GENRE" tags are written to audiofiles
+ cueprint/cuetag: no "DATE" and "GENRE" tags are read/written from/to
+ audiofiles
Alexander (lxandr)
description: updated
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "cuetag.patch" 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
Launchpad Janitor (janitor) wrote :

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

Changed in cuetools (Ubuntu):
status: New → Confirmed
Revision history for this message
Ville Pilvio (vpilvio) wrote :

2.5 years, and the problem is still ongoing. No year, no genre

Revision history for this message
shao.lo (shao-lo) wrote :

The patch is slightly out of date now (new version attached), but concept absolutely works! Why not push this out?

Revision history for this message
vlad (jozmi) wrote :

This patch mostly works.
However, for some reason using the mentioned hack with sed it adds empty space line after the text which ends up with GENRE field having space or empty line after text. Can this empty space be removed.

Revision history for this message
shao.lo (shao-lo) wrote :

I'm no expert with sed, but if it is a problem with white space you could try updating the sed command to also call the tr command as follows to remove space from either end of DATE and GENRE.

DATE=`sed -n 's!^REM DATE "\?\([^"]*\)"\?!\1!p' "$cue_file" | tr -d '[:space:]'`
GENRE=`sed -n 's!^REM GENRE "\?\([^"]*\)"\?!\1!p' "$cue_file" | tr -d '[:space:]'`

Revision history for this message
Danil (varrah) wrote :

Trying to ping Sven on this: https://github.com/svend/cuetools/issues/36

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.