All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] runqueue: Fix missing fakeworker under dry run
@ 2016-05-09 13:15 Richard Purdie
  2016-05-09 16:36 ` Christopher Larson
  0 siblings, 1 reply; 2+ messages in thread
From: Richard Purdie @ 2016-05-09 13:15 UTC (permalink / raw)
  To: bitbake-devel

We shouldn't try and use fakeworker when performing a dry_run. This
makes the core match the other fakeworker execution points.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>

diff --git a/bitbake/lib/bb/runqueue.py b/bitbake/lib/bb/runqueue.py
index 0af4678..8d2aa7b 100644
--- a/bitbake/lib/bb/runqueue.py
+++ b/bitbake/lib/bb/runqueue.py
@@ -2095,7 +2095,7 @@ class RunQueueExecuteScenequeue(RunQueueExecute):
             bb.event.fire(startevent, self.cfgData)
 
             taskdep = self.rqdata.dataCache.task_deps[fn]
-            if 'fakeroot' in taskdep and taskname in taskdep['fakeroot']:
+            if 'fakeroot' in taskdep and taskname in taskdep['fakeroot'] and not self.cooker.configuration.dry_run:
                 if not self.rq.fakeworker:
                     self.rq.start_fakeworker(self)
                 self.rq.fakeworker.stdin.write("<runtask>" + pickle.dumps((fn, realtask, taskname, True, self.cooker.collection.get_file_appends(fn), None)) + "</runtask>")




^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] runqueue: Fix missing fakeworker under dry run
  2016-05-09 13:15 [PATCH] runqueue: Fix missing fakeworker under dry run Richard Purdie
@ 2016-05-09 16:36 ` Christopher Larson
  0 siblings, 0 replies; 2+ messages in thread
From: Christopher Larson @ 2016-05-09 16:36 UTC (permalink / raw)
  To: Richard Purdie; +Cc: bitbake-devel

[-- Attachment #1: Type: text/plain, Size: 482 bytes --]

On Mon, May 9, 2016 at 6:15 AM, Richard Purdie <
richard.purdie@linuxfoundation.org> wrote:

> We shouldn't try and use fakeworker when performing a dry_run. This
> makes the core match the other fakeworker execution points.
>
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
>

Looks good.
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics

[-- Attachment #2: Type: text/html, Size: 938 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-05-09 16:37 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-09 13:15 [PATCH] runqueue: Fix missing fakeworker under dry run Richard Purdie
2016-05-09 16:36 ` Christopher Larson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.