diff --git a/lib/core/libimagrt/src/runtime.rs b/lib/core/libimagrt/src/runtime.rs index a51fe8c5..b0b3c8a8 100644 --- a/lib/core/libimagrt/src/runtime.rs +++ b/lib/core/libimagrt/src/runtime.rs @@ -52,7 +52,6 @@ pub struct Runtime<'a> { configuration: Option, cli_matches: ArgMatches<'a>, store: Store, - use_pipe_magic: bool, stdin_is_tty: bool, stdout_is_tty: bool, } @@ -139,15 +138,12 @@ impl<'a> Runtime<'a> { Store::new(storepath, &config) }; - let pipe_magic = matches.is_present(Runtime::pipe_magic_name()); - store_result.map(|store| { Runtime { cli_matches: matches, configuration: config, rtp: rtp, store: store, - use_pipe_magic: pipe_magic, stdout_is_tty: ::atty::is(::atty::Stream::Stdout), stdin_is_tty: ::atty::is(::atty::Stream::Stdin), } @@ -236,13 +232,6 @@ impl<'a> Runtime<'a> { .takes_value(true) .value_name("LOGDESTS")) - .arg(Arg::with_name(Runtime::pipe_magic_name()) - .long(Runtime::pipe_magic_name()) - .short("P") - .help("Use pipe-detection. With this flag, imag expects a JSON store on STDIN if stdin is not a TTY and prints the store to STDOUT if it is not a TTY.") - .required(false) - .takes_value(false)) - } /// Get the argument names of the Runtime which are available @@ -327,11 +316,6 @@ impl<'a> Runtime<'a> { "logging-destinations" } - /// Get the argument name for pipe magic - pub fn pipe_magic_name() -> &'static str { - "pipe-magic" - } - /// Initialize the internal logger fn init_logger(matches: &ArgMatches, config: Option<&Value>) { use log::set_max_level; @@ -460,10 +444,10 @@ impl<'a> Runtime<'a> { } pub fn stdout(&self) -> OutputProxy { - if self.use_pipe_magic && !self.stdout_is_tty { - OutputProxy::Err(::std::io::stderr()) - } else { + if self.stdout_is_tty { OutputProxy::Out(::std::io::stdout()) + } else { + OutputProxy::Err(::std::io::stderr()) } } @@ -472,10 +456,10 @@ impl<'a> Runtime<'a> { } pub fn stdin(&self) -> Option { - if self.use_pipe_magic && !self.stdin_is_tty { - None - } else { + if self.stdin_is_tty { Some(::std::io::stdin()) + } else { + None } }