(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.

Support multi- and single- file modes.

Fix r1932 (i told you it was untested).

Factor out the flac(1)-running part to run_flac, which is identical
in both versions.

WTF!  I just noticed that i forgot the part of hhlp that would
actually create the array reference, but it worked anyway!  Now i
find out perl has autovivified these since at least 2000.  Yet i'm
sure they weren't being autovivified when i had to do something
similar at Lexmark a couple years ago.  Maybe it was a bug in
whatever Perl version i was using, or more likely i was just smoking
crack...

Kill hhlp.

Untested: make multi-file the default with a single-file option.

Here's my hacked up multi-file support.  Needs a lot of cleanup,
and needs to support single-file mode as well.

(newjob): Should be setting $SIG{CHLD} to 'DEFAULT', not 'IGNORE'.
The former happened to work on NetBSD but isn't right, and doesn't
work on Linux 2.6 (was causing system(flac) to return -1).

Make MusicBrainz querying optional (untested).

Oops, support for untagged files was not complete.  Fix.

Update for version 3.

Update CDDEV explanation to talk about MusicBrainz; this was left over
from when this was a zsh script that tried to figure that out itself.

Lose mkcue and require MusicBrainz::Client >= 0.09 (for release dates).

Add support for reading release dates from MusicBrainz.

Thanks to Matthias Friedrich <matt@mafr.de> for adding this to
MusicBrainz::Client::Simple and suggesting it.

Looks like MusicBrainz::Client exposed the TOC info i needed all
along; stop running mkcue and use that instead.

Thanks to Matthias Friedrich <matt@mafr.de> for the suggestion and
demo code.

Add vorbis comment documentation URLs.

Add DISCNUMBER support.

Single-track support broke multi-track.  Fix both.

Match tag names case-insensitively.

Add notes for recent changes.

(flac2mp3): Force track number argument to int.

Don't stupidly take only one argument; take multiple files.

Support single-track FLAC files (no cue sheet).

If the TRACKNUMBER tag is present, use it on the first MP3 file; this
should only be set on a single-track FLAC file.

Don't blow up if any of the tags we look for are not defined; just
don't use them.

(get_tags): http://us.xiph.org/ogg/vorbis/doc/v-comment.html says the
keys are supposed to be case insensitive.

Support multi-artist flac files.

1. Switch to TITLE[n] style tags
2. Support multi-artist discs

(tags_file): Split all the duplicated candidate-tags file writing
stuff from tags into this new function.  It uses TITLE[n] style tags,
and ARTIST[n] tags for multi-artist discs.

(tags): Use tags_file.  If any of the matching MusicBrainz album
objects' has_various_artists function returns true, make a
candidate-tags-0v file with blank ARTIST[n] entries, as well as
putting them in the candidate-tags-$i file.

Document changes for version 2.

(get_tags): Fix TITLE regexp; it was mistakenly not using ^ like the
others.  Also, as follow-up to r1733, document why we don't match = .

(run_or_die): Print command if verbose mode.

(flac2mp3): Pass --quiet and --verbose options to flac and lame as
appropriate.  If --lame-options given, also pass those to lame.
Previously --preset standard was hard-coded, now it is just the
default if --lame-options is not given.

(MAIN): Now take options: --lame-options, -q/--quiet, -v/--verbose,
and -h/--help.

Work around cperl-mode scanning bug.

Add debug mode.

Add support for a post-processor hook.

(fa-flacd):
    Document the hook.

    (flac): Execute and remove the hook if it exists before removing
    the temporary directory.

(fa-rip):
    (make_post_processor): New function to create the post-processor
    file, if defined.

    (MAIN): Take new -p/--post-processor option.

Need the g (global) flag on that substitution operation.

Copy run_or_die from fa-rip and use that to run flac | lame.