| Message ID | 20250617171332.3162295-3-ross.burton@arm.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/4] oeqa/utils/command: fast-path get_bb_var() | expand |
diff --git a/meta/lib/oeqa/selftest/cases/esdk.py b/meta/lib/oeqa/selftest/cases/esdk.py index 9f5de2cde7f..7a5fe00a082 100644 --- a/meta/lib/oeqa/selftest/cases/esdk.py +++ b/meta/lib/oeqa/selftest/cases/esdk.py @@ -27,11 +27,7 @@ class oeSDKExtSelfTest(OESelftestTestCase): return glob.glob(pattern)[0] @staticmethod - def run_esdk_cmd(env_eSDK, tmpdir_eSDKQA, cmd, postconfig=None, **options): - if postconfig: - esdk_conf_file = os.path.join(tmpdir_eSDKQA, 'conf', 'local.conf') - with open(esdk_conf_file, 'a+') as f: - f.write(postconfig) + def run_esdk_cmd(env_eSDK, tmpdir_eSDKQA, cmd, **options): if not options: options = {} if not 'shell' in options:
Discovered when looking for users of get_bb_var()'s postconfig argument, this wrapper around runCmd() has a postconfig argument that has odd behaviour: it _appends_ the new configuration to local.conf instead of having them used for this specific run (unlike the other functions in commands.py) None of the eSDK tests use this functionality, so remove it. Future test cases that need to write further configuration should do so directly. Signed-off-by: Ross Burton <ross.burton@arm.com> --- meta/lib/oeqa/selftest/cases/esdk.py | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-)