matthiasbeyer/imag
1
0
Fork 0
Code Releases Activity

List diaries only if output is not a pipe

Browse source 
Beforehand, this command:

    imag diary -d foo list | cat

listed each entry twice because the reporting feature of libimagrt
forwarded the list to stdout and the normal output was written to
stderr.

With this patch, we get the output only once in this case.

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>
This commit is contained in:
Matthias Beyer 2019-07-21 10:42:27 +02:00
parent d53e306459
commit 03fec5f4db
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
bin/domain/imag-diary/src/list.rs
View file

@ -57,7 +57,9 @@ pub fn list(rt: &Runtime) {
.for_each(|id| { .for_each(|id| {
let _ = rt.report_touched(&id).unwrap_or_exit(); let _ = rt.report_touched(&id).unwrap_or_exit();
writeln!(rt.stdout(), "{}", id).to_exit_code().unwrap_or_exit() if !rt.output_is_pipe() {
writeln!(rt.stdout(), "{}", id).to_exit_code().unwrap_or_exit()
}
}); });
} }