#458956 abcde: Flac single file support does not embed track information

#458956#5
Date:
2008-01-03 21:20:48 UTC
From:
To:
I see that it is deliberate in the flac format that embedded Cue sheets do
not have track information like track names embedded in them:
http://flac.sourceforge.net/faq.html#general__no_cuesheet_tags

OK, whatever.  However, given that abcde has all of the information already
and in fact generates an external cue file with artist and track names, I
think the information should be put *somewhere* even if it's just in random
comment tags in the flac file.  As it is right now, I end up with two files:

foo.flac.cue - Cue sheet with artist and track name info
foo.flac - Has cue sheet with only track info

If I:
metaflac --remove-tag=CUESHEET foo.flac
metaflac --import-cuesheet-from=foo.flac.cue foo.flac
metaflac --export-cuesheet-to=foo.flac.cue2 foo.flac
then foo.flac.cue2 is missing the extra information in foo.flac.cue

Arguably this is a problem with metaflac - maybe metaflac
--import-cuefile-from should (optionally?) write auxilary track info to
comments, but I don't know what the right answer is.  The more straight
forward answer would seem to be that abcde should manually insert tags for
each track; I don' tknow what the format of this would be however.

#458956#10
Date:
2008-01-04 15:41:22 UTC
From:
To:
Unfortunately there doesn't seem to be a real standard for this type of
comment.  http://www.gophernet.org/articles/vorbiscomment/ suggests
SUBTITLE[TRACK 1]=Track 1 name
but there's no mention of the obvious SUBARTIST for multi-artist albums.
Other tag-standard-related pages:
http://www.xiph.org/vorbis/doc/v-comment.html
http://reactor-core.org/ogg-tagging.html

#458956#15
Date:
2008-01-08 20:42:57 UTC
From:
To:
Here's my hacky patch to add SUBTITLE tags to flac files when ripping
in 1-file mode and when a cue sheet is being generated.  Since abcde
seems unmaintained I havent put a lot of effort into doing things the
right way.  Existing flac files can be tagged using existing cue files
outside of the context of abcde with something like this:
 awk -v r='"' '/^[ \t][ \t]*TRACK/ {printf "SUBTITLE(TRACK %s)=",$2}       \
               /^[ \t][ \t]*TITLE/ {split($0, st, r); printf "%s\n",st[2]}' \
        < foo.flac.cue | metaflac --import-tags-from=- foo.flac

#458956#20
Date:
2008-01-09 17:31:04 UTC
From:
To:
Oops, I used SUBTITLE() rather than SUBTITLE[] in the patch.  Easy
enough to fix by hand....

#458956#25
Date:
2010-04-25 10:49:33 UTC
From:
To:
G'day,

I was having a look at the patch that was generated by Chris Chiappa,
and started to put it into my own copy of the script.  While I was
looking, I decided to try and find out what the -M option actually did,
and noticed a rather strange condition.

In line 50, the comment is "Create a CUE  file".  Setting this *should*
work.  However...

Parsing of options starts at line 3095.  As expected, M is there (line
3119) and sets the DOCUE variable to y.  This looks fine... But.

Line 3239 is the determination of actions which are to be done from the
ACTIONS variable, which is set via the -a command.  You may note in line
3252 that DOCUE is reset to n, *regardless* of whether it has been
previously set!  This, I have found, is the issue at stake.  Commenting
this line out, and running the suggested `abcde -1 -M -o flac` command
gave me an flac file with embedded tags.  Finally!

Hopefully this will help other people who have come across this error,
and not known why it was the case.  It would be a much simpler patch
than what Chris worked out, and I'll look at doing that myself later.

Regards,

Russell.