New top-level package: wrench

This tiny package provides simple way for any part of Juju to check if a "wrench should be dropped in the works" at some part of Juju's execution. It checks $datadir/wrench for files which signal that something should break, performing some error checking and logging.

The idea is that judicious calls to wrench.IsEnabled can be permanently left in the code to aid manual and CI testing.


Ok, so the scanner takes care of DOS and Unix line endings just fine. I just wrote up a little example to make sure. I'm not sure what your concerns about DOS line endings are about. Please let me know if I've missed something.

http://play.golang.org/p/Iciha8LdXH  

My comments about TrimSpace only made sense for the previous implementation that didn't use bufio.Scanner. TrimSpace is still useful however in case someone accidentally adds some spaces around lines.

You're right that I need to be calling Err() on the scanner and logging if something went wrong during file reading. I'll fix that.