untested error handling changes

Oops, really unify, and also fix SubprocessError to consume the stderr stream.

unify subprocess handling

Implement (and tested) do_write.

make the do_write stub actually work (missing imports and typos)

Don't save TRACKNUMBER.

some write work

Add filename building.

Obviously haven't run this in a while; forgot to remove my dead taglib module...

don't throw Exception from do_read

Expand SubprocessError to report the failing command, and test it.

Oops, abs(status) in SubprocessError.

add note about abuse of Tags

Start on a script to help rewrite tags en masse.

Seems I screw up every time I get a CD with non-ascii titles, and I'm tired of
hacking up a fix with interactive shell-scripting...

some demo apic reading code; would be nice to save all APIC frames.

rename

add tool for stripping tags

simple reindent

reorder includes

rejigger this to a stand-alone tool rather than a py module

When I switched from my crappy taglib module to using lame to add album art
(r162), I introduced a bug when album art is not present in the FLAC.  Try to
fix it.

UNTESTED

note how stupid lame is

Add --id3v2-latin1. Default in newer lame is to transcode to UTF-16! Quite
wasteful on most of my files, and I don't have any that won't work fine in
latin1, so let's use that.

specify --pad-id3v2-size 0; it seems to make no difference, even though it
looks to me like the default in the code is 128

lame supports adding album art directly, it's just not very well documented
(still not in the man page in latest release), so switch to that instead of my
crappy taglib module

Switch from --add-id3v2 to --id3v2-only (sadly, lame still adds old id3 tags
anyway)

Don't fetch same ASIN image twice.

This is all working again, even multi-artist records.

don't fetch images for asin=None

Don't make so many calls to Musicbrainz web service.

Also add --discid option for testing without involving physical discs.

Some of this code had rotted.

Write ALBUMARTIST field and adapt to some API changes.

Copy ALBUMARTIST from FLAC file to TPE2 in ID3.

/usr/bin/python

+ look into http://www.sacredchao.net/quodlibet/wiki/Development/Mutagen

(flac_pictures): Fix compile warnings about description signedness.

Finalize release notes.

doc fix

Add missing option documentation.

(main): Write tempdir to stderr so user doesn't have to hunt for it.

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=439790

(MSF_OFFSET): Add global for 150 frames.
(mkcue): Don't skip all of the first gap, but only MSF_OFFSET frames.

Patch by: Martin Michlmayr

(run_flac): Only add --picture option if cover.front exists.
(flac): Keep unlinking even if unlink(cover.front) fails.

Suggested by: Martin Michlmayr

Only flac2mp3 needs taglib.

From Martin Michlmayr:
Fix path for org.diplodocus

Move flac.so and taglib.so to flac_archive package.

Bah, go ahead and require flac 1.1.3 .

Oops, forgot this.

Since mp3 has no FEATURING tag, append those to ARTIST.

Pick minor style nit.

Use flac_archive.tags.Tags, greatly simplifying things.  Put PART tag
in flac filename, as with flac2mp3 does for mp3 filename.

Make use of Tags.gets's new separator argument.

Move Tags class into new flac_archive package, fix a bug in it, and
test it.  These stupid scripts will finally start sharing code and
getting some formal testing.

flac_archive/tags.py:
    (Tags): Move here from flac2mp3, and rename ._tags to ._tracks .
    The tag-loading loop from flac2mp3 get_tags becomes the new .load
    method.

    (Tags.get): Return all matching tag data instead of either global
    or per-track (oops).

    (Tags.gets): Adapt to that change, and allow other separators than
    just '\n', which becomes a default.

    (Tags.all, Tags.track): New methods for getting a tag block for
    all tags (whole disc) or just a track, respectively.  To be used
    by fa-flacd in single-file and per-track modes, respectively.

flac2mp3:
    Use flac_archive.tags.Tags.  Use Tags.load in get_tags .

setup.py:
    Note MIT license and install flac_archive package.

test/run:
    Copy test-runner from py-gurps.

test/test.py:
    Load all tests.

test/testtags.py:
    Test Tags class.

s/'''/"""/

pointless quote change

Dump svn:keywords .

All files have at least one track.

Fix pics nonsense.

(flac2mp3): Fix skip_until processing: stop pointless splitting the
list into two variables and other weird stuff!

(Tags): Storing the global tags in the same dict meant that __len__
was incorrect (+1) for multi-track files.  Keep them separate.

Support my old, numberless TITLE tags.

(flac2mp3): Never set date to 'unknown'.

(Tags.set): Need to upcase the key when setting, too.

1.1.3 has been out a while; uncomment --picture support.

Bah unicode.

Add hack to pull down musicbrainz info without having a disc; useful
when you're away from home and rip WAVs into itunes...  This should be
a separate program.

Update CHANGES, list another dependency in README.

more

TODO

s/--export-vc-to/--export-tags-to/

Hm, why just copy front cover PICTUREs from flac to mp3?  The flac
PICTURE metadata block has the exact same metadata as the mp3 ID3v2
APIC frame; just copy them all.

flac.c:
    (flac_pictures): Return fields in the same order they're found in
    the FLAC format.  Convert the description to a unicode object
    rather than returning a UTF-8 encoded str object.

taglib.cc:
    (taglib_apic): Take a list of picture tuples and add them all as
    APIC frames.

    (add_apic_frame_to_mp3): Take the list of picture tuples and pass
    them straight to taglib_apic.

flac2mp3:
    (flac2mp3): Take a list of picture tuples instead of just one;
    pass them all to add_apic_frame_to_mp3.

    (find_pics): Rename from find_pic.  Save all PICTURE metadata
    blocks rather than just saving the first front cover picture.

Set encoding of description.

(flac2mp3): Oops, keep an unmunged version of the output filename,
too.

Copy flac PICTURE to mp3 APIC using new extension modules.

Don't hard-code front cover type; let the caller choose that.

Replace crappy demo code for flac PICTURE and mp3 APIC support with
halfway-decent (i hope) Python extension modules.

Rejigger this a bit; no longer segfaults.

Add stupid diff to metaflac to extract PICTURE.

Here's some demo code i whipped up to add APIC to an mp3 with taglib.

Force ID3v2 tags; when i add ID3v2 APIC, iTunes starts ignoring the
ID3v1 stuff, guess i don't blame it...

Add (commented-out) flac 1.1.3 PICTURE support.

Grab Amazon artwork, but don't do anything with it yet.

Oops, in multi-file mode, all tracks are ripped even with -t option.
Fixing this also helps lose some duplication.

Use album.release where possible, restrict
musicbrainz2.webservice.ReleaseIncludes to tracks.

(flac2mp3): Don't need to int(track) now that the loop in main does it.

(Tags): New class that should be moved somewhere shared so fa-flacd
and others can use it.  Simplifies keeping track of per-disc and
per-track tags.

(get_tags): Build and return a Tags object instead of mucking about
with different tags in inconsistent (and broken!) ways.

(main): Oy, main needs to be split up.  Change the main loop to use
the Tags object, fix discnum-is-a-str-not-int bug, make track always
an int here (previously was sometimes str), correctly handle multiple
tracks per flac, and append PART to TITLE if present.

(flac): UNTESTED: Put disc number at the front of the output file name,
not the reader (?!).

(flacloop): Don't flip out in multi-file mode if 'cue' doesn't exist.

(tags_file): Hey, Off-by-one, so kind of you to visit.  Go away!

Rewrite in python (single-file and multi-artist modes not tested).

Here's another silly setup.py.

Rewrite in python.

Move to a good home.

Add Python implemention of Jobs package.

Fix a job in Jobs::run (@jobs < $maxjobs, not <=) and icky icky
copy-paste it into flac2mp3, which now supports parallelization.

This is now tested; both multi-file and single-file modes work fine,
and multi-file is now fully parallelizable.

(flac): Refactor this to encode one wav file to one flac, instead of
looping over all the wavs for a multi-file mode.  This leaves behind
the fa-rip directories instead of cleaning them up; i left a comment
where the cleanup should happen; easy enough to fix.  When creating or
removing files or directories, test for $!{EEXIST} and $!{ENOENT} on
failure rather than stat(2)ing up front.

(flacloop): Move some of the multi-file processing here into our new
Jobs::run get-jobs function; it manages a queue of wav files so we can
fully parallelize even multi-file jobs, instead of having the disc as
the unit of parallelization.

Improve Jobs package:
 - no longer need global @jobs and @finished
 - @finished is gone completely
 - new run function which takes care of all the details; you just give
   it a function to call to get a new job function, and optional
   reportingi functions.

(flacloop): Oops, remove leftover $SIG{CHLD} setting (still UNTESTED).

UNTESTED: Refactor the job parallelization stuff into a generic Jobs package.

Here's a simple hack to split single cue-containing flac files into
single-track flac files.  It's almost entirely a copy/paste of
parts of fa-flacd and flac2mp3; all the common parts really should
be factored out...

Store TRACKNUMBER tag in multi-file mode (oops).

Fix two minor errors.

Fill in release date (today) now that i've tested r1926.

update

Drop the single quotes in printing post-processor args; they may
contain quotes anyway.

Use better identifers and add comments.