From b25ebbc408444e55922a568ad90ebc2491e70177 Mon Sep 17 00:00:00 2001 From: Chris Laplante Date: Wed, 6 Aug 2025 10:53:06 -0400 Subject: [PATCH] bitbake: cooker: remove some redundant control flow Prior to commit aa84a900e ("cooker: Ensure delays are accurately transfered to the idle loops from runqueue"), this was necessary. But now retval is returned directly. (Bitbake rev: c2eb4336fe10b1bf8bbc6291c32ca362840f39e1) Signed-off-by: Chris Laplante Signed-off-by: Richard Purdie --- bitbake/lib/bb/cooker.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/bitbake/lib/bb/cooker.py b/bitbake/lib/bb/cooker.py index 9e4000b63c..fb87368f17 100644 --- a/bitbake/lib/bb/cooker.py +++ b/bitbake/lib/bb/cooker.py @@ -1434,8 +1434,7 @@ You can also remove the BB_HASHSERVE_UPSTREAM setting, but this may result in si if quietlog: bb.runqueue.logger.setLevel(rqloglevel) return bb.server.process.idleFinish(msg) - if retval is True: - return True + return retval self.idleCallBackRegister(buildFileIdle, rq) @@ -1504,8 +1503,6 @@ You can also remove the BB_HASHSERVE_UPSTREAM setting, but this may result in si bb.event.disable_heartbeat() return bb.server.process.idleFinish(msg) - if retval is True: - return True return retval self.reset_mtime_caches()