Print out the internal command error message when make_dep_file fails
Fix random test (It just had a bad comment at the top...)
Cleanup old cruft
Remove <base/time.h>, moving all uses to std::chrono

Also update some related things (TTimerFd) to use std::chrono while passing through.

orly/indy/manager_base.h: Changed fairly significantly in order to migrate to chrono. Restructured minimally to try to keep things fairly relatable / readable.
orly/indy/disk/durable_manager.h: I did some moderate refactoring to reduce duplication
Change JHM to use std::chrono rather than the old C struct for time (timespec)
util/time: Add another useful function
Pull out SleepUntil to be a helper function doing the check first optimization

At least at one point in the past the optimization was added because it was a significant perf win.
orly/indy/manager_base: Remove an excess system call
util/time: Update some documentation, move functions to better names
Add a cpu_clock which provides a chrono compatible clock wrapping CLOCK_THREAD_CPUTIME_ID

Requirements: http://en.cppreference.com/w/cpp/concept/TrivialClock
Update <base/timer.h> to use std::chrono

And update all users.

Pulls in a rather large bit of making <orly/perf/kv_exercise> compile again
Remove manual clocks / hand-rolled timekeeping (timespec) code

This moves it all to std::chrono which provides much cleaner facilities for working with time data and pushes the units into the C++ type system.
Use a helper function to convert chrono durations to seconds represented by a double.

Normalizes two different directions and a couple one-offs
Address reviewer comments

 - Add IfLt0 check
 - Remove unnecessary std:: (And a few more cases)
 - Remove unintended header comment change
Merge pull request #241 from cmaloney/rm-base-time

Move towards std::chrono (Remove <base/time>, cleanup in JHM)
orly/client/repl: Add missing include
JHM: use clang
Use clang to compile stig packages

It's more particular about unused variables. So silence it there.

Conflicts:
	orly/compiler.cc
Ignore unused private field warning for sabot tests

The test intentionally uses it to test packing/unpacking of private fields.
Ignore tautological compare warning