diff options
author | Moritz Poldrack <git@moritz.sh> | 2023-03-04 10:56:48 +0100 |
---|---|---|
committer | Robin Jarry <robin@jarry.cc> | 2023-03-07 00:13:18 +0100 |
commit | a0318de57f4884e6df76cbe74acf6221fccaace3 (patch) | |
tree | db38596f26effccb6fd87f523bec55ad76ba52d1 | |
parent | d3dea73aa53791a94f86c31689a10880fabeb84c (diff) | |
download | aerc-a0318de57f4884e6df76cbe74acf6221fccaace3.tar.gz |
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 <git@moritz.sh>
Acked-by: Robin Jarry <robin@jarry.cc>
-rw-r--r-- | CHANGELOG.md | 1 | ||||
-rw-r--r-- | 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) } |