Set encoding of description.

[flac-archive] / flac2mp3

diff --git a/flac2mp3 b/flac2mp3
index 5fc05ed65065e50d56b0a585d637702fd99d3af8..b4ce99eff1c127166ebab67913f100a09ad49553 100755 (executable)
--- a/flac2mp3
+++ b/flac2mp3
@@ -55,12 +55,13 @@ 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
 
 ################################################################################
 # The child processes
 
-def flac2mp3(fn, title, artist, album, date, track, skip_until):
+def flac2mp3(fn, title, artist, album, date, track, skip_until, pic=None):
     (title, artist, album, date) = [(x == None and 'unknown') or x
                                     for x in (title, artist, album, date)]
     try:
@@ -83,17 +84,24 @@ def flac2mp3(fn, title, artist, album, date, track, skip_until):
     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)]
 
-    outfile = ('%s (%s) %02d %s.mp3' % (artist, album,
-                                        track, title)).replace('/', '_')
+    quoted_outfile = ('%s (%s) %02d %s.mp3' % (artist, album,
+                                               track, title)).replace('/', '_')
 
     run_or_die(3, "flac %s -cd %s %s '%s' | lame --add-id3v2 %s --tt '%s' --ta '%s' --tl '%s' --ty '%s' --tn %d - '%s'"
                % (flac_options, skip_arg or '', until_arg or '', fn,
-                  lame_options, title, artist, album, date, track, outfile))
+                  lame_options, title, artist, album, date, track,
+                  quoted_outfile))
+
+    if pic != None:
+        taglib.add_apic_frame_to_mp3(outfile, pic[0], pic[1], pic[2])
 
     return 0
 
@@ -176,7 +184,7 @@ def get_tags(fn):
 
     tags = Tags()
 
-    p = Popen(['metaflac', '--export-vc-to=-', fn], stdout=PIPE)
+    p = Popen(['metaflac', '--export-tags-to=-', fn], stdout=PIPE)
     for line in (x.rstrip() for x in p.stdout):
         (tag, value) = line.split('=', 1)
 
@@ -193,6 +201,18 @@ def get_tags(fn):
 
     return tags
 
+def find_pic(fn, tags):
+    pic = tags.get('__flac2mp3_PICTURE')
+
+    if not isinstance(pic, tuple):
+        for i in flac.get_pictures(fn):
+            if i[1] == flac.PICTURE_TYPE_FRONT_COVER:
+                pic = i[:3]
+                break
+        tags.set('__flac2mp3_PICTURE', pic)
+
+    return pic
+
 def main(argv):
     # Control the exit code for any uncaught exceptions.
     try:
@@ -255,7 +275,7 @@ def main(argv):
                                  tags.gets('ARTIST', track),
                                  album,
                                  tags.gets('DATE', track),
-                                 track, args[i]])
+                                 track, args[i], find_pic(fn, tags)])
                     track = i + 2
             except Exception, error:
                 sys.stderr.write(getattr(error, 'msg', ''))