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.

Make handling of child process death consistent.

(get_tags): Just match /TITLE/ instead of /TITLE=/, to support the
TITLE[1] .. TITLE[n] tag style.

Note GNU make requirement.

Remove debugging bits accidentally committted.

Add  URL keyword.

Make Getopt::Long behave sanely, as i do elsewhere.

Use Getopt::Long and Pod::Usage instead of Getopt::Std.  As a side
effect, support long options.  Add OPTIONS documentation section.

Collapse fa-rip and fa-tags into a single Perl program, fa-rip.
Fix up the documentation.

Support a -t track-count option to fa-rip and mkcue, so we can skip
stupid data tracks.

No more .toc file now that we use mkcue.

Ah-hah!  I have finally found the bug that caused my mp3s to include a
bit of the next track at the end of one, but only sometimes.  The
hundredths of a second vs. sector was certainly a bug, but was not
this bug.

(get_decode_args): Use sprintf %02d for each field.  Previously i
was just stringing them together with colon and dot, so "9" becomes
"9" instead of "09", which flac sees as 90!

(get_decod_args): The third time field is *sectors*, not hundredths of
a second.  One second is 75 sectors.

Don't count tracks twice.

Use mkcue to make cue sheets, instead of cdrdao.

Sanitize filename sanitization and quoting.  I'm not sure why i was
s/[:'"]/_/g, momentary insanity i suppose.  All i need do is replace /
with _ on output filenames and replace ' with '\'' on filenames that
will go to the shell surrouned by '' quotes, instead of being passed
directly into a program's argv.

fa-flacd:
    (flac): Do / sanitization only, since we're not going through the
    shell, but running flac directly.

flac2mp3:
    (flac2mp3): We weren't sanitizing the input filename; start doing
    so.  Only do ' sanitization for input file and output id3tag
    information, additionally do / sanitization on $outfile.

(get_tags): Properly handle errors from metaflac.

(flac): Whoops, need to use sanitized version in some more places.

(flac): Copy filename sanitization from flac2mp3.