- unused variables removed
- docs updated to reflect reality

Makes the tests a bit more comprehensible.

Rename some RUN= names to be clearer.

* Better naming
* hook script is more general
* tests/README.md

This works by moving the pre-volume-hook sligtly later. One effect
of this is that it can happen after the device hook when previously
it happened before. This is probably not very important but it does
have a knock-on effect on one of the tests.

This avoids locale issues when parsing rsync output.

fixes #111

"previously declared as a struct; this is valid, but may result in linker errors under the Microsoft C++ ABI [-Werror,-Wmismatched-tags]"

If we're given a device name of an LVM internal layer, we shouldn't
just ignore the layer name and snapshot something else. Instead,
assume something's gone horribly wrong and fail hard.

Also tweak some diagnostics and comments, and add new tests for this
and the not-a-LV scenario.

On several systems, udevadm has been observed not to give output for
some LVs for no terribly obvious reason, causing rsbackup-snapshot-hook
to refuse to work with those LVs (#72).

dmsetup also lets us canonicalise device names to work around the
original issue which caused udevadm to be invoked (#23), but is closer
to the source of truth, and thus more likely to be reliable than
udevadm.

As a side effect of this rework, VG names containing hyphens should
now be handled more convincingly.

And fix the errors thus brought to light (all in the test suite,
fortunately).

This probably explains why the test suite didn't pick up 7b84f914
back in 2015.

(While I'm here, add a couple of --must-run to t-hook-fsck-fail that
ought to have been there.)

This is a (currently undocumented) feature that allows a backup
to be excluded from consideration as a link destination, for
instance if it is no longer trusted for some reason.

/usr/share/doc/rsbackup/ was ending up with both rsbackup-manual.html
and rsbackup-manual.in.html, which was untidy.

fixed #91

fixes #101

re #105

fix #105. Unclear why things work/failed where they did.

fixes #106

fixes #98

Quadratic behavior reduced to n.log n. Better is surely available but
this will do for now.

(Distinguish from Render::Context...)

Not actually very useful given the reports we actually generate.

...since that's what all the production callers use anyway.