X-Git-Url: https://diplodocus.org/git/flac-archive/blobdiff_plain/d8b1024f3b57a8619fa0e07029e9205e760a84d7..1ae4fb3e6c8b5fe43ae5b5f50b2a8e1368a37f42:/flac2mp3 diff --git a/flac2mp3 b/flac2mp3 index 5ebc873..d9f9f7b 100755 --- a/flac2mp3 +++ b/flac2mp3 @@ -1,6 +1,6 @@ -#! /usr/bin/env python2.4 +#!/usr/bin/python -''' +""" =head1 NAME B - transcode FLAC file to MP3 files @@ -16,6 +16,10 @@ may be the kind of FLAC file B generates. That is, it contains a cue sheet, one TITLE tag per track listed therein, and ARTIST, ALBUM, and DATE tags. +Note that lame is retarded, and parses B directly itself! So, in order +for it to transcode textual tags, you must specify the encoding in LANG, e.g. +LANG=en_US.utf-8 + =head1 OPTIONS =over 4 @@ -48,20 +52,23 @@ Written by Eric Gillespie . =cut -''' #' # python-mode is sucks +""" -import re, sys, traceback +import os, re, sys, tempfile, traceback from optparse import OptionParser from subprocess import Popen, PIPE import org.diplodocus.jobs -from org.diplodocus import flac, taglib from org.diplodocus.util import run_or_die +from flac_archive import flac +from flac_archive.tags import Tags + ################################################################################ # The child processes -def flac2mp3(fn, title, artist, album, date, track, skip_until, pics=None): +def flac2mp3(fn, title, artist, album_artist, album, discnum, date, + track, skip_until, pics=None): (title, artist, album) = [(x == None and 'unknown') or x for x in (title, artist, album)] if date == None: @@ -82,24 +89,43 @@ def flac2mp3(fn, title, artist, album, date, track, skip_until, pics=None): verbose and tmp.append('--verbose') lame_options = ' '.join(tmp) - outfile = ('%s (%s) %02d %s.mp3' % (artist, album, - track, title)).replace('/', '_') - # Escape any single quotes ' so we can quote this. - (fn, title, artist, - album, date) = [x.replace("'", r"'\''") - for x in (fn, title, artist, album, date)] + (fn, title, artist, album_artist, + album, date) = [(x or '').replace("'", r"'\''") + for x in [fn, title, artist, album_artist, album, date]] - quoted_outfile = ('%s (%s) %02d %s.mp3' % (artist, album, - track, title)).replace('/', '_') + album_artist_options = '' + if album_artist: + album_artist_options = "--tv 'TPE2=%s'" % album_artist - run_or_die(3, "flac %s -cd %s '%s' | lame --add-id3v2 %s --tt '%s' --ta '%s' --tl '%s' --ty '%s' --tn %d - '%s'" - % (flac_options, ' '.join(skip_until), fn, - lame_options, title, artist, album, date, track, - quoted_outfile)) + outfile_album = album + discnum_options = '' + if discnum != None: + outfile_album = '%s (disc %s)' % (album, discnum) + discnum_options = "--tv 'TPOS=%d'" % int(discnum) - if pics != None: - taglib.add_apic_frame_to_mp3(outfile, pics) + quoted_outfile = ('%s (%s) %02d %s.mp3' % (artist, outfile_album, + track, title)).replace('/', '_') + + pic_options = None + if pics: + (fd, picfn) = tempfile.mkstemp() + f = os.fdopen(fd, 'wb') + f.write(pics[0][7]) + f.close() + pic_options = "--ti '%s'" % picfn + try: + run_or_die(3, "flac %s -cd %s '%s' | lame --id3v2-only --id3v2-latin1 --pad-id3v2-size 0 %s --tt '%s' --ta '%s' --tl '%s' --ty '%s' --tn %d %s %s %s - '%s'" + % (flac_options, ' '.join(skip_until), fn, + lame_options, title, artist, album, date, track, + pic_options, album_artist_options, + discnum_options, quoted_outfile)) + finally: + if pic_options: + try: + os.unlink(picfn) + except: + pass return 0 @@ -112,7 +138,8 @@ def tformat(m, s, c): def get_decode_args(fn): l = [] - p = Popen(['metaflac', '--export-cuesheet-to=-', fn], stdout=PIPE) + p = Popen(['metaflac', '--no-utf8-convert', '--export-cuesheet-to=-', fn], + stdout=PIPE) for line in (x.rstrip() for x in p.stdout): m = re.search(r'INDEX 01 (\d\d):(\d\d):(\d\d)$', line) if m != None: @@ -146,59 +173,16 @@ def get_decode_args(fn): return args -# XXX other things should usue this; flac files, for example, should -# get PART as part of the filelname, same as mp3s. -class Tags(object): - def __init__(self): - self._global = {} - self._tags = {} - def __len__(self): - return len(self._tags) - def get(self, key, track=None): - key = key.upper() - try: - try: - return self._tags[track][key] - except KeyError: - return self._global[key] - except KeyError: - return None - def gets(self, key, track=None): - value = self.get(key, track) - if value == None: - return None - return '\n'.join(value) - def set(self, key, value, track=None): - key = key.upper() - if track == None: - tags = self._global - else: - try: - tags = self._tags[track] - except KeyError: - tags = self._tags[track] = {} - if key not in tags: - tags[key] = [] - tags[key].append(value) - def get_tags(fn): - '''Return the ARTIST, ALBUM, and DATE tags followed by the TITLE tags - in the file FN.''' + """Return the ARTIST, ALBUM, and DATE tags followed by the TITLE tags + in the file FN.""" tags = Tags() - p = Popen(['metaflac', '--export-tags-to=-', fn], stdout=PIPE) - for line in (x.rstrip() for x in p.stdout): - (tag, value) = line.split('=', 1) - - m = re.search(r'\[([0-9]+)]$', tag) - if m != None: - tag = tag[:m.start()] - track = int(m.group(1)) - else: - track = None + p = Popen(['metaflac', '--no-utf8-convert', '--export-tags-to=-', fn], + stdout=PIPE) + tags.load(p.stdout) - tags.set(tag, value, track) # XXX dataloss! check status status = p.wait() @@ -227,6 +211,7 @@ def main(argv): traceback.print_exc() return 2 + separator = ' ' try: global debug, flac_options, lame_options, quiet, verbose debug = options.debug @@ -240,14 +225,10 @@ def main(argv): args = get_decode_args(fn) tags = get_tags(fn) - album = tags.gets('ALBUM') + album = tags.gets('ALBUM', separator=separator) discnum = tags.gets('DISCNUMBER') track = tags.gets('TRACKNUMBER') - # lame doesn't seem to support disc number. - if discnum != None: - album = '%s (disc %s)' % (album, discnum) - # Stupid hack: only a single-track file should have the # TRACKNUMBER tag, so use it if set for the first pass through # the loop. At the end of the loop, we'll set $track for the @@ -260,15 +241,21 @@ def main(argv): pics = flac.get_pictures(fn) for i in range(len(tags)): - title = tags.gets('TITLE', track) + title = tags.gets('TITLE', track, separator) part = tags.gets('PART', track) if part != None: title = '%s - %s' % (title, part) - jobs.append([fn, title, - tags.gets('ARTIST', track), - album, + version = tags.gets('VERSION', track) + if version != None: + title = '%s (%s)' % (title, version) + artist = tags.get('ARTIST', track) + artist.extend(tags.get('FEATURING', track)) + album_artist = tags.gets('ALBUMARTIST', track) + jobs.append((fn, title, + ', '.join(artist), + album_artist, album, discnum, tags.gets('DATE', track), - track, args[i], pics]) + track, args[i], pics)) track = i + 2 except Exception, error: sys.stderr.write(getattr(error, 'msg', ''))