From a0318de57f4884e6df76cbe74acf6221fccaace3 Mon Sep 17 00:00:00 2001 From: Moritz Poldrack Date: Sat, 4 Mar 2023 10:56:48 +0100 Subject: ipc: stay silent if command succeeded It would be nicer if aerc behaved the same way most other tools behave in that no news is good news. Signed-off-by: Moritz Poldrack Acked-by: Robin Jarry --- CHANGELOG.md | 1 + lib/ipc/send.go | 4 +--- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index b81765bf..cc229447 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -29,6 +29,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/). - The standard Usenet signature delimiter `"-- "` is now prepended to `signature-file` and `signature-cmd` if not already present. - All `aerc(1)` commands now interpret `aerc-templates(7)` markup. +- running commands (like mailto: or mbox:) no longer prints a success message ### Deprecated diff --git a/lib/ipc/send.go b/lib/ipc/send.go index 522e944a..4b09268c 100644 --- a/lib/ipc/send.go +++ b/lib/ipc/send.go @@ -37,9 +37,7 @@ func ConnectAndExec(args []string) error { } // TODO: handle this in a more elegant manner - if resp.Error == "" { - fmt.Println("result: success") - } else { + if resp.Error != "" { fmt.Println("result: ", resp.Error) } -- cgit v1.2.3-54-g00ecf