1
0
mirror of https://git.yoctoproject.org/poky synced 2026-06-02 01:19:52 +00:00

bitbake: bitbake: Rename setscene enforce filtering variable

In line with the inclusive language migration defined at:

https://wiki.yoctoproject.org/wiki/Inclusive_language

rename:

BB_SETSCENE_ENFORCE_WHITELIST -> BB_SETSCENE_ENFORCE_IGNORE_TASKS

(Bitbake rev: 2e243ac06581c4de8c6e697dfba460ca017d067c)

(Bitbake rev: f8f7b80a0df4646247e58238a52a7d85a37116d4)

Signed-off-by: Scott Murray <scott.murray@konsulko.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Scott Murray
2022-02-13 16:06:51 -05:00
committed by Richard Purdie
parent 18a400d117
commit 3c971c0400
3 changed files with 20 additions and 19 deletions
+1
View File
@@ -39,6 +39,7 @@ bitbake_renamed_vars = {
"BB_HASHBASE_WHITELIST": "BB_BASEHASH_IGNORE_VARS", "BB_HASHBASE_WHITELIST": "BB_BASEHASH_IGNORE_VARS",
"BB_HASHCONFIG_WHITELIST": "BB_HASHCONFIG_IGNORE_VARS", "BB_HASHCONFIG_WHITELIST": "BB_HASHCONFIG_IGNORE_VARS",
"BB_HASHTASK_WHITELIST": "BB_TASKHASH_IGNORE_TASKS", "BB_HASHTASK_WHITELIST": "BB_TASKHASH_IGNORE_TASKS",
"BB_SETSCENE_ENFORCE_WHITELIST": "BB_SETSCENE_ENFORCE_IGNORE_TASKS",
} }
def infer_caller_details(loginfo, parent = False, varval = True): def infer_caller_details(loginfo, parent = False, varval = True):
+17 -17
View File
@@ -386,8 +386,8 @@ class RunQueueData:
self.warn_multi_bb = False self.warn_multi_bb = False
self.multi_provider_whitelist = (cfgData.getVar("MULTI_PROVIDER_WHITELIST") or "").split() self.multi_provider_whitelist = (cfgData.getVar("MULTI_PROVIDER_WHITELIST") or "").split()
self.setscenewhitelist = get_setscene_enforce_whitelist(cfgData, targets) self.setscene_ignore_tasks = get_setscene_enforce_ignore_tasks(cfgData, targets)
self.setscenewhitelist_checked = False self.setscene_ignore_tasks_checked = False
self.setscene_enforce = (cfgData.getVar('BB_SETSCENE_ENFORCE') == "1") self.setscene_enforce = (cfgData.getVar('BB_SETSCENE_ENFORCE') == "1")
self.init_progress_reporter = bb.progress.DummyMultiStageProcessProgressReporter() self.init_progress_reporter = bb.progress.DummyMultiStageProcessProgressReporter()
@@ -2107,9 +2107,9 @@ class RunQueueExecute:
if task is not None: if task is not None:
(mc, fn, taskname, taskfn) = split_tid_mcfn(task) (mc, fn, taskname, taskfn) = split_tid_mcfn(task)
if self.rqdata.setscenewhitelist is not None: if self.rqdata.setscene_ignore_tasks is not None:
if self.check_setscenewhitelist(task): if self.check_setscene_ignore_tasks(task):
self.task_fail(task, "setscene whitelist") self.task_fail(task, "setscene ignore_tasks")
return True return True
if task in self.tasks_covered: if task in self.tasks_covered:
@@ -2501,11 +2501,11 @@ class RunQueueExecute:
self.scenequeue_updatecounters(task) self.scenequeue_updatecounters(task)
def sq_check_taskfail(self, task): def sq_check_taskfail(self, task):
if self.rqdata.setscenewhitelist is not None: if self.rqdata.setscene_ignore_tasks is not None:
realtask = task.split('_setscene')[0] realtask = task.split('_setscene')[0]
(mc, fn, taskname, taskfn) = split_tid_mcfn(realtask) (mc, fn, taskname, taskfn) = split_tid_mcfn(realtask)
pn = self.rqdata.dataCaches[mc].pkg_fn[taskfn] pn = self.rqdata.dataCaches[mc].pkg_fn[taskfn]
if not check_setscene_enforce_whitelist(pn, taskname, self.rqdata.setscenewhitelist): if not check_setscene_enforce_ignore_tasks(pn, taskname, self.rqdata.setscene_ignore_tasks):
logger.error('Task %s.%s failed' % (pn, taskname + "_setscene")) logger.error('Task %s.%s failed' % (pn, taskname + "_setscene"))
self.rq.state = runQueueCleanUp self.rq.state = runQueueCleanUp
@@ -2568,8 +2568,8 @@ class RunQueueExecute:
#bb.note("Task %s: " % task + str(taskdepdata).replace("], ", "],\n")) #bb.note("Task %s: " % task + str(taskdepdata).replace("], ", "],\n"))
return taskdepdata return taskdepdata
def check_setscenewhitelist(self, tid): def check_setscene_ignore_tasks(self, tid):
# Check task that is going to run against the whitelist # Check task that is going to run against the ignore tasks list
(mc, fn, taskname, taskfn) = split_tid_mcfn(tid) (mc, fn, taskname, taskfn) = split_tid_mcfn(tid)
# Ignore covered tasks # Ignore covered tasks
if tid in self.tasks_covered: if tid in self.tasks_covered:
@@ -2583,7 +2583,7 @@ class RunQueueExecute:
return False return False
pn = self.rqdata.dataCaches[mc].pkg_fn[taskfn] pn = self.rqdata.dataCaches[mc].pkg_fn[taskfn]
if not check_setscene_enforce_whitelist(pn, taskname, self.rqdata.setscenewhitelist): if not check_setscene_enforce_ignore_tasks(pn, taskname, self.rqdata.setscene_ignore_tasks):
if tid in self.rqdata.runq_setscene_tids: if tid in self.rqdata.runq_setscene_tids:
msg = ['Task %s.%s attempted to execute unexpectedly and should have been setscened' % (pn, taskname)] msg = ['Task %s.%s attempted to execute unexpectedly and should have been setscened' % (pn, taskname)]
else: else:
@@ -3070,12 +3070,12 @@ class runQueuePipe():
print("Warning, worker left partial message: %s" % self.queue) print("Warning, worker left partial message: %s" % self.queue)
self.input.close() self.input.close()
def get_setscene_enforce_whitelist(d, targets): def get_setscene_enforce_ignore_tasks(d, targets):
if d.getVar('BB_SETSCENE_ENFORCE') != '1': if d.getVar('BB_SETSCENE_ENFORCE') != '1':
return None return None
whitelist = (d.getVar("BB_SETSCENE_ENFORCE_WHITELIST") or "").split() ignore_tasks = (d.getVar("BB_SETSCENE_ENFORCE_IGNORE_TASKS") or "").split()
outlist = [] outlist = []
for item in whitelist[:]: for item in ignore_tasks[:]:
if item.startswith('%:'): if item.startswith('%:'):
for (mc, target, task, fn) in targets: for (mc, target, task, fn) in targets:
outlist.append(target + ':' + item.split(':')[1]) outlist.append(target + ':' + item.split(':')[1])
@@ -3083,12 +3083,12 @@ def get_setscene_enforce_whitelist(d, targets):
outlist.append(item) outlist.append(item)
return outlist return outlist
def check_setscene_enforce_whitelist(pn, taskname, whitelist): def check_setscene_enforce_ignore_tasks(pn, taskname, ignore_tasks):
import fnmatch import fnmatch
if whitelist is not None: if ignore_tasks is not None:
item = '%s:%s' % (pn, taskname) item = '%s:%s' % (pn, taskname)
for whitelist_item in whitelist: for ignore_tasks in ignore_tasks:
if fnmatch.fnmatch(item, whitelist_item): if fnmatch.fnmatch(item, ignore_tasks):
return True return True
return False return False
return True return True
+2 -2
View File
@@ -154,12 +154,12 @@ class RunQueueTests(unittest.TestCase):
self.shutdown(tempdir) self.shutdown(tempdir)
def test_setscenewhitelist(self): def test_setscene_ignore_tasks(self):
with tempfile.TemporaryDirectory(prefix="runqueuetest") as tempdir: with tempfile.TemporaryDirectory(prefix="runqueuetest") as tempdir:
cmd = ["bitbake", "a1"] cmd = ["bitbake", "a1"]
extraenv = { extraenv = {
"BB_SETSCENE_ENFORCE" : "1", "BB_SETSCENE_ENFORCE" : "1",
"BB_SETSCENE_ENFORCE_WHITELIST" : "a1:do_package_write_rpm a1:do_build" "BB_SETSCENE_ENFORCE_IGNORE_TASKS" : "a1:do_package_write_rpm a1:do_build"
} }
sstatevalid = "a1:do_package a1:do_package_qa a1:do_packagedata a1:do_package_write_ipk a1:do_populate_lic a1:do_populate_sysroot" sstatevalid = "a1:do_package a1:do_package_qa a1:do_packagedata a1:do_package_write_ipk a1:do_populate_lic a1:do_populate_sysroot"
tasks = self.run_bitbakecmd(cmd, tempdir, sstatevalid, extraenv=extraenv) tasks = self.run_bitbakecmd(cmd, tempdir, sstatevalid, extraenv=extraenv)