summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Purdie <richard.purdie@linuxfoundation.org>2013-05-30 12:28:05 +0000
committerRichard Purdie <richard.purdie@linuxfoundation.org>2013-06-28 10:01:01 +0100
commit89c8493d4d85044cd72af2756569d15e87cd5947 (patch)
treea0772c4d02ecaa7c5b030efba60be07683b3e11e
parent7950a307bc7d4104e6cfb09bb2ea267c5da83f2a (diff)
downloadopenembedded-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.bbclass4
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()