diff --git a/bin/core/imag/src/main.rs b/bin/core/imag/src/main.rs index 23b80785..f866794c 100644 --- a/bin/core/imag/src/main.rs +++ b/bin/core/imag/src/main.rs @@ -421,8 +421,5 @@ fn forward_commandline_arguments(m: &ArgMatches, scmd: &mut Vec) { push(Some("ignore-ids"), Runtime::arg_ignore_ids_name(), m , scmd); - - push(None , Runtime::arg_logdest_name() , m , scmd); - } diff --git a/lib/core/libimagrt/src/logger.rs b/lib/core/libimagrt/src/logger.rs index 2b71d577..cf4f975e 100644 --- a/lib/core/libimagrt/src/logger.rs +++ b/lib/core/libimagrt/src/logger.rs @@ -280,8 +280,8 @@ fn translate_destinations(raw: &Vec) -> Result> { fn aggregate_global_destinations(matches: &ArgMatches, config: Option<&Value>) -> Result> { - match config { + None => Ok(vec![LogDestination::default()]), Some(cfg) => cfg .read("imag.logging.destinations") .map_err(Error::from) @@ -293,21 +293,6 @@ fn aggregate_global_destinations(matches: &ArgMatches, config: Option<&Value>) Error::from(err_msg(msg)) }) .and_then(translate_destinations), - None => { - if let Some(values) = matches.value_of(Runtime::arg_logdest_name()) { - // parse logdest specification from commandline - - values.split(",") - .fold(Ok(vec![]), move |acc, dest| { - acc.and_then(|mut v| { - v.push(translate_destination(dest)?); - Ok(v) - }) - }) - } else { - Ok(vec![ LogDestination::default() ]) - } - } } } diff --git a/lib/core/libimagrt/src/runtime.rs b/lib/core/libimagrt/src/runtime.rs index 1fa0429f..5fb367dc 100644 --- a/lib/core/libimagrt/src/runtime.rs +++ b/lib/core/libimagrt/src/runtime.rs @@ -245,13 +245,6 @@ impl<'a> Runtime<'a> { .required(false) .takes_value(true)) - .arg(Arg::with_name(Runtime::arg_logdest_name()) - .long(Runtime::arg_logdest_name()) - .help("Override the logging destinations from the configuration: values can be seperated by ',', a value of '-' marks the stderr output, everything else is expected to be a path") - .required(false) - .takes_value(true) - .value_name("LOGDESTS")) - .arg(Arg::with_name(Runtime::arg_ignore_ids_name()) .long(Runtime::arg_ignore_ids_name()) .help("Do not assume that the output is a pipe to another imag command. This overrides the default behaviour where imag only prints the IDs of the touched entries to stdout if stdout is a pipe.") @@ -344,11 +337,6 @@ impl<'a> Runtime<'a> { self } - /// Get the argument name for the logging destination - pub fn arg_logdest_name() -> &'static str { - "logging-destinations" - } - #[cfg(feature = "pub_logging_initialization")] pub fn init_logger(matches: &ArgMatches, config: Option<&Value>) { Self::_init_logger(matches, config)