vbauerster / mpb

multi progress bar for Go cli applications
The Unlicense
2.29k stars 123 forks source link

Fix a deadlock with Bar.Abort(true) #101

Closed mtrmac closed 2 years ago

mtrmac commented 2 years ago

Don't block Bar.serve on Progress.serve, because Progress.serve blocks on Bar.serve (e.g. in Bar.forceRefreshIfLastUncompleted or bar.wSyncTable).

Fixes #100 .

vbauerster commented 2 years ago

@mtrmac Any update? I'm actually ok to merge this one with aforementioned patch.

mtrmac commented 2 years ago

@mtrmac Any update? I'm actually ok to merge this one with aforementioned patch.

I apologize about the late response, your version works fine as well. Thanks!