`Box<Error + Sync + Send>` is far more common than `Box<Error +
Send>` - common conveniences like conversion from strings are not
implemented for `Box<Error + Send>`, `Box<Error + Send>` cannot be
stored in `io::Error`s, etc.

This is a breaking change, but non-`Sync` errors should be rare so the
fallout shouldn't be too bad.