diff options
author | Tim Culverhouse <tim@timculverhouse.com> | 2023-04-25 07:57:13 -0500 |
---|---|---|
committer | Robin Jarry <robin@jarry.cc> | 2023-04-26 00:07:50 +0200 |
commit | 8c2a9cf6b90d5fe18fa7e5e110f6460e967ee036 (patch) | |
tree | 4b943ad0d4a1959fd8e8c2f28bf1afdcce321f69 /go.mod | |
parent | a72313b8a1c5d1b6df646e839e7c2a27697f4c88 (diff) | |
download | aerc-8c2a9cf6b90d5fe18fa7e5e110f6460e967ee036.tar.gz |
term: fix handling of failed command
A panic occurs when a terminal is launched with a command that fails:
:term blabla
The underlying terminal does not need to be closed if the command failed
to start: all resources are cleaned up upon failure to start already.
Don't attempt to close terminal if the command didn't start.
With the above fix in place, the tab will linger around until a redraw
occurs, as there is nothing queuing a redraw on this behavior. Add a
QueueRedraw in the tabs.Remove method to clean up the tabbar.
The issue in tcell-term that causes this panic has been address there as
well, released as 0.7.1. Update aerc's version.
Reported-by: Koni Marti <koni.marti@gmail.com>
Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
Tested-by: Koni Marti <koni.marti@gmail.com>
Acked-by: Robin Jarry <robin@jarry.cc>
Diffstat (limited to 'go.mod')
-rw-r--r-- | go.mod | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -3,7 +3,7 @@ module git.sr.ht/~rjarry/aerc go 1.18 require ( - git.sr.ht/~rockorager/tcell-term v0.7.0 + git.sr.ht/~rockorager/tcell-term v0.7.1 git.sr.ht/~sircmpwn/getopt v1.0.0 github.com/ProtonMail/go-crypto v0.0.0-20230417170513-8ee5748c52b5 github.com/arran4/golang-ical v0.0.0-20230318005454-19abf92700cc |