This equality looks a bit odd, how about `if !odir.exists()`?

It would also be nice to have a check for `if !odir.is_dir() { early_error(...) }`