Make Scaladoc actually exit with non-zero exit code in case of errors, as its docs say it does

I did this work for Twitter, on Twitter's time, so I'm covered under Twitter's CLA. 

Fixes https://issues.scala-lang.org/browse/SI-8885.