Merge branch 'master' of git.distorted.org.uk:~mdw/publish/public-git/dvdrip

* 'master' of git.distorted.org.uk:~mdw/publish/public-git/dvdrip:
  dvd-info.c: Trigger the easy lookup if explicit chapter bounds aren't given.
  dvd-info.c: Don't forget to dump the last titleset.
  dvd-info.c: Add missing `\' in output banner.

chkdvdimg.c: Open the image file with `open_dvd'.

This means that it will wait until we actually insert a disc.  Yay!

lib.c, and knock-on: Add MODE argument to `open_dvd'.

Everyone wants read-only for now.

dvd-info.c: Trigger the easy lookup if explicit chapter bounds aren't given.

dvd-info.c: Don't forget to dump the last titleset.

dvd-info.c: Add missing `\' in output banner.

Add a new program `dvd-info' to report on interesting things.

I'm getting deeper in here, I know.

checkimg: Don't complain about every mismatch.

We can tell from the summary.

dvd-sector-copy.c: Use `bad_hi' consistently for the upper bound when done.

It's just conceivable that `good' is less than `bad_hi' at this point,
which screws everything up.  It's essential to pick a consistent upper
bound here.

dvd-sector-copy.c, lib.[ch]: Improve and publish the number parsing functions.

Makefile: Factor out the `CFLAGS'.

Makes overriding from the command line less awful.

dvd-sector-copy.c: Show file ends in the dump.

dvd-sector-copy.c: Wrap long line properly.

dvdrip-upload: Fix self-call the case where there's no daemon running.

dvd-sector-copy.c: Truncate the event list before processing a lost `stop'.

dvd-sector-copy.c: Report final statistics correctly.

dvd-sector-copy.c: Dump the event queue before we start processing it.

dvd-sector-copy.c: Eliminate unused variable.

dvd-sector-copy.c: Fix botched debugging code.

chkdvdimg.c: Name the file when reporting too many trailing zeros.

dvd-sector-copy.c: Add missing handler for the `-i' option.

chkdvdimg.c: Retrieve device size using the new function.

... which works on block devices as well as ordinary files.

dvd-sector-copy.c: Export device-size machinery to the library.

dvdrip: Check for the device size apparently changing during the copy.

This indicates something going horribly wrong.

dvdrip: Fail if we can't read the `.IFO' hash for the disc id.

This may help with the kernel reporting stale medium sizes.

dvd-sector-copy.c: Add option to verify source id.

scripts: Call utilities via environment variables.

dvd-id.c, lib.[ch]: Improve DVD ID computation.

Move the code to the library.  Accept flags (to the library function and
the utility) to fail if portions of the ID can't be determined.

Also, improve `dvd-id' to print IDs for multiple disc images.

chkdvdimg: Add a new program for checking DVD images.

Primarily I'm interested in checking that the length is right, because
one of my DVD drives sometimes reports the wrong medium size
(specifically, I think, the size for the previous medium).  Also,
sometimes there are zero sectors stuck on the end, which complicates
finding the backup UDF anchor descriptor, so provide an option to trim
these.

lib.h: Publish a `MEMCMP' macro, like `STRCMP'.

lib.c, lib.h: Introduce `vmoan_syserr' and `moan_syserr'.

This cleans up this little family of functions.  There's a behaviour
change: `vmoan' now prints a newline which it didn't before, and callers
have been adjusted to compensate.

lib.c: Remove redundant format attributes.

Should use `PRINTF_LIKE' nowadays anyway.

dvdrip-retry-botched-vobs: Retry using the newly determined block list.

Fortunately, I didn't actually engage this case when I did this to my
collection.

dvdrip-retry-botched-vobs: Process discs in a useful order.

dvdrip-retry-botched-vobs: Eject discs when we're done with them.

dvdrip-retry-botched-vobs: Delete the retry file correctly.

dvdrip-retry-botched-vobs: Be more vocal about wrong discs.

New programs for dealing with VOBs ripped without being decrypted.

When my drives were configured with the wrong region, `libdvdcss' would
have to crack the keys the hard way, and it sometimes failed.  I mostly
shrugged my shoulders, but now I've fixed this problem, I thought I'd
track down the miscopied VOBs and make some tools for fixing the
problem.

dvdrip-monitor: New program to keep track of what's going on.

dvdrip: Leave a note in the working directory about which device we're using.

dvd-sector-copy.c, lib.[ch]: Move the file hacking functions to the library.

Makefile: Only `dvd-sector-copy' needs `-lm'.

multiprogress.c: Leave the cursor at the start of the last line.

Things still go wrong when stray characters get echoed but it's less
bad.  I think.

multiprogress.c: Probe termcap/info for newline sequence.

Just to show willing.  Provide the obvious default, because it seems to
be necessary!

multiprogress.c: Provide the obvious default for `cr'.

dvd-cache-keys.c: Allow multiple devices and/or image files.

dvd-cache-keys.c: Move the existence check into `kick_vob'.

Much neater.

dvd-cache-keys.c: Pass index and titleness as unsigned.

dvd-sector-copy.c: Allow averaging `alpha' to be set as a parameter.

dvd-sector-copy.c: Reinterpret the `step' as being beyond the high bound.

dvd-sector-copy.c: Just a light reformatting.

dvd-sector-copy.c: Go back to showing just disc progress when continuing.

dvd-sector-copy.c: Remove the pointless `status' variable.

Relic of former days when `dvdrip' used the return code to infer that
there were bad blocks.  Nowadays it checks for the `badblocks' file.

dvd-sector-copy.c: Report the number of bad sectors in the bad-blocks file.

A big mess of changes all at once.

I'm usually quite good about factoring out all of the changes I make, but
in this case it doesn't seem worth the bother.

dvdrip: Squish some code a bit.

dvdrip: Abolish the affordances for using `dvdbackup'.

dvdrip: Add `-r' option to retry the failed blocks.

dvd-sector-copy.c (report_progress): Fiddle with the averaging parameter.

dvd-sector-copy.c (report_progress): Show a throbber to help gauge speed.

dvd-sector-copy.c (recovered): Don't print comment if outside a VOB file.

dvd-sector-copy.c, dvdrip: Cue prompt to check image from `badblocks' file.

The exit status from the final run isn't helpful, but the bad-blocks
file is.

dvdrip: Prepare the `dest' file early, so that it can be edited.

dvd-sector-copy.c: Use the correct character to match comments.

Oops.

sort-badblocks: Another handy testing utility.

dvd-sector-copy.c: Limit `recovery_read' return value to length requested.

Otherwise the caller gets properly confused.

dvd-sector-copy.c: Add more debugging about the recovery algorithm.

dvdrip-upload: Make script sexecutable, like it should be.

dvd-sector-copy.c: Secret option to dump the exact bad sectors encountered.

I don't think this one quite works properly yet.

dvd-sector-copy.c: Include file-relative information in bad-blocks file.

Also, tolerate these comments when reading them back in.

dvd-sector-copy.c: Read into the correct part of the buffer.  Again.

dvd-sector-copy.c: Remove broken field-width specifier.

I failed to include space for the decimal point and fraction, so it
didn't actually do anything.  And it turns out that I'm happy with it
like this.

dvd-sector-copy.c: Factor out some file-hacking machinery.

dvdrip: Collect together bad-blocks lists from repeated runs.

checkimg, distill: A couple of scripts useful for testing and debugging.

dvd-sector-copy.c: Leave progress report visible before debugging.

dvd-sector-copy.c: Fix formatting for debugging code.

dvd-sector-copy.c: Dump buffer after final recovery.

dvd-sector-copy.c: Don't dump fake-bad-block search if no bad blocks.

dvd-sector-copy.c: Fix final recovery.

Write this the other way around, because it's easier to read that way.
And then notice that it's broken because it doesn't actually set the
return value properly.

dvd-sector-copy.c: Write new data into the correct part of the buffer.

The debugging message had the right value, which made this annoyingly
hard to track down.

dvd-sector-copy.c: Check more carefully for overflow conditions.

dvd-sector-copy.c: Check `n' nonnegative before comparing with unsigned.

Otherwise it might be coerced to unsigned, at which point the comparison
means something completely different.

dvd-sector-copy.c: Adjust the base position when shifting.

Fixes stupid bug.

dvd-sector-copy.c: Fix formatting for `ssize_t'.

dvd-sector-copy.c: Rewrite the bad-sector recovery machinery.

It's now much more aggressive about skipping good sectors squished
between bad ones.

dvd-sector-copy.c: Extract the post-recovery code into a function.

The `goto' is ugly.

dvd-sector-copy.c: Sink `report_bad_blocks_progress' towards recovery.

dvd-sector-copy.c: Make `read_sectors' more persistent after short reads.

It now continues trying to read until it gets an error, end-of-file, or
it completes the task requested of it.  For consistency, we must also
report an error after a short read forced by a synthetic bad sector,
which requires a little additional machinery.

dvd-sector-copy.c: Move `i' to the debug-only variables block.

dvd-sector-copy.c: Split bad-block recovery out into its own function.

dvd-sector-copy.c: Add machinery for installing pretend bad sectors.

For testing the recovery algorithm(s).

dvd-sector-copy.c: Force return to start of line on progress output.

dvd-sector-copy.c: Merge adjacent retry ranges when reading from a file.

dvd-sector-copy.c, dvdrip: Make input and output files positional args.

They're not really optional.

dvd-sector-copy.c: Hoist out the common sector-range parsing machinery.

dvd-sector-copy.c: Make `outfd' be global variable.

dvd-sector-copy.c: Hoist `carefully_write' up near the system utilities.

dvd-sector-copy.c: Split up the things which used to be in `struct source'.

dvd-sector-copy.c: Eliminate `struct source'; just make the things global.