diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-05-30 12:28:05 +0000 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-06-28 10:01:01 +0100 |
commit | 89c8493d4d85044cd72af2756569d15e87cd5947 (patch) | |
tree | a0772c4d02ecaa7c5b030efba60be07683b3e11e | |
parent | 7950a307bc7d4104e6cfb09bb2ea267c5da83f2a (diff) | |
download | openembedded-core-89c8493d4d85044cd72af2756569d15e87cd5947.tar.gz openembedded-core-89c8493d4d85044cd72af2756569d15e87cd5947.tar.bz2 openembedded-core-89c8493d4d85044cd72af2756569d15e87cd5947.zip |
package: Ensure we iterate all the pool objects
There is the possibility that if we don't iterate through the multiprocessing
pool objects we might not catch return codes and this could lead to hung/zombie
processes either temproarily or on a wider scale. Adding this certainly doesn't
hurt anything and is better practise so we might as well do it.
Its not 100% clear if this fixes some issues or not.
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
-rw-r--r-- | meta/classes/package.bbclass | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/meta/classes/package.bbclass b/meta/classes/package.bbclass index f25f5671e5..8e6029a0a3 100644 --- a/meta/classes/package.bbclass +++ b/meta/classes/package.bbclass @@ -906,7 +906,7 @@ python split_and_strip_files () { import multiprocessing nproc = multiprocessing.cpu_count() pool = bb.utils.multiprocessingpool(nproc) - processed = pool.imap(oe.package.runstrip, sfiles) + processed = list(pool.imap(oe.package.runstrip, sfiles)) pool.close() pool.join() @@ -1265,7 +1265,7 @@ python package_do_filedeps() { import multiprocessing nproc = multiprocessing.cpu_count() pool = bb.utils.multiprocessingpool(nproc) - processed = pool.imap(oe.package.filedeprunner, pkglist) + processed = list(pool.imap(oe.package.filedeprunner, pkglist)) pool.close() pool.join() |