add C++ build rules

add more tests apparently written on May 13 2013

found some older tests too, still laughable...

set mode on output file, and don't clobber existing

more cleanup

more error check

some cleanup

fix for flacs with no embedded art

submitted an untested fix way back in Thu 2012-11-01 15:52:39 -0700
and only just now tested (and fixed) it; oops

untested attempt to add error checking

untested because, oh look, i never submitted any build rules!

add stupid hack for checking missing files

here's a never-submitted bug-fix to tags.make_filename

added tags.make_filename for rewrite-tags on 2012-12-26

yet it didn't work; apparently i made this fix locally back then and never
submitted it...

Fix bugs:

- read: support a tag appearing with the same value on a subset of tracks and
  nowhere else; previously this was ok if the value were differnet, but if say
  only a few tracks had VERSION=radio and the rest had no VERSION, VERSION
  would erroneously become a global tag

- write: support each track having own ALBUM with on global ALBUm (e.g. most
  tracks are 'album=foo' but a few are 'album=foo extended'); previously broke
  as ALBUm had to be global

- bonus: i noticed write was also missing ALBUMARTIST; added

make get_path_safe return None when no tag value

drop unused, strange make_filename returning just the album (wtf?); doesn't
work if no global album anyway

Hmm, some standardization sitting around in a working copy since November...

Use ALBUMARTIST as filename if available, fallback to ARTIST.

Remove variable 'outfile', unused since I switched from my crappy taglib
module to using lame to add album art at Tue 2012-07-24 14:52:43 -0700

Use TPOS to store discnumber in mp3 files.

add --no-utf8-convert to flac2mp3 and flacsplit metaflac runs

copy VERSION values into mp3 title, in parens

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 (?!).