aboutsummaryrefslogtreecommitdiffstats
path: root/config/parse.go
diff options
context:
space:
mode:
authorRobin Jarry <robin@jarry.cc>2024-01-27 21:36:25 +0100
committerRobin Jarry <robin@jarry.cc>2024-01-31 16:08:45 +0100
commitfd4dd42408856048dd71f83ae1f30e2ab84621da (patch)
tree6ae89fff21d1100d814249545170b7d572bb8740 /config/parse.go
parent4998406cce859791013880ec5f8f402007a2d1bd (diff)
downloadaerc-fd4dd42408856048dd71f83ae1f30e2ab84621da.tar.gz
ipc: wait for app to be ready before accepting commands
Pass a cancellable context to the IPC server and defer accepting connections until the first worker message has been received by the main thread. This is not a real fix for the :split command not working at startup. :split requires a mail store to be present on the currently selected account tab+folder. I could have waited for a types.DirectoryContents message but it seems silly and bug prone since such a message may never arrive and we don't want to pool IPC messages forever. Also, a types.DirectoryContents message may arrive for one account but :split is only effective *per account* and the default selected account tab is the first one. In any case, this band aid prevents aerc from crashing or breaking down the terminal when running IPC commands in the aerc-startup hook. Fixes: https://lists.sr.ht/~rjarry/aerc-devel/%3CCYPN7AVYQ69S.WV0T67VM6WX3%40wegtam.com%3E Fixes: https://todo.sr.ht/~rjarry/aerc/173 Reported-by: Jens Grassel <jens@wegtam.com> Reported-by: Justine Smithies <justine@smithies.me.uk> Signed-off-by: Robin Jarry <robin@jarry.cc> Tested-by: Thomas Böhler <witcher@wiredspace.de> Reviewed-by: Thomas Böhler <witcher@wiredspace.de>
Diffstat (limited to 'config/parse.go')
0 files changed, 0 insertions, 0 deletions