Jose Quaresma <[email protected]> writes: >> Transition to ThreadPoolExecutor (eb6a6820928472ef194b963b606454e731f9486f) >> broke he >> >> | Checking sstate mirror object availability: ... >> >> progress bar because the removed 'thread_worker' was still referenced >> in an asynchronous function. As the result of the future is never >> read, the resulting backtrace is silently discarded. > > I have tested the patch at the time and the progress bar works as expected, > maybe something breaks after that.
Probably, you saw only the full 100% bar produced by 'bb.event.ProcessFinished' But the patch does | - def checkstatus(thread_worker, arg): | + def checkstatus(arg): | ... | bb.event.fire(bb.event.ProcessProgress(msg, len(tasklist) - thread_worker.tasks.qsize()), d) This will always raise an exception and 'bb.event.ProcessProgress' will never be fired. > Can the num_tasks_done be moved to there and call the first > next(num_tasks_done) so we can remove the +1 on the ProcessProgress I will send a v2 Enrico
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#173418): https://lists.openembedded.org/g/openembedded-core/message/173418 Mute This Topic: https://lists.openembedded.org/mt/95064126/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
