Don't assume color=always when explicitally specified

Fixes #31546
This commit is contained in:
Steven Allen 2016-02-10 20:02:20 -05:00
parent 5d771cd5b3
commit 03ef55b1c8

View File

@ -934,17 +934,17 @@ pub fn build_session_options(matches: &getopts::Matches) -> Options {
Some("human") => ErrorOutputType::HumanReadable(color),
Some("json") => ErrorOutputType::Json,
None => ErrorOutputType::default(),
None => ErrorOutputType::HumanReadable(color),
Some(arg) => {
early_error(ErrorOutputType::default(), &format!("argument for --error-format must \
be human or json (instead was \
`{}`)",
arg))
early_error(ErrorOutputType::HumanReadable(color),
&format!("argument for --error-format must be human or json (instead \
was `{}`)",
arg))
}
}
} else {
ErrorOutputType::default()
ErrorOutputType::HumanReadable(color)
};
let unparsed_crate_types = matches.opt_strs("crate-type");