osmith has submitted this change. ( https://gerrit.osmocom.org/c/python/osmo-python-tests/+/36934?usp=email ) Change subject: scripts/osmotestconfig: remove unused rmtemp arg ...................................................................... scripts/osmotestconfig: remove unused rmtemp arg The rmtmp arg to test_all_apps() is always False, remove it. Change-Id: I469d854cda0a23bbf545953ad2b94666df8e7d04 --- M scripts/osmotestconfig.py 1 file changed, 13 insertions(+), 2 deletions(-) Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve diff --git a/scripts/osmotestconfig.py b/scripts/osmotestconfig.py index 8b3bb84..a62edd7 100755 --- a/scripts/osmotestconfig.py +++ b/scripts/osmotestconfig.py @@ -157,7 +157,7 @@ def test_all_apps(apps, app_configs, tmpdir="writtenconfig", verbose=True, - confpath=".", rmtmp=False, ignore_configs=[]): + confpath=".", ignore_configs=[]): check_configs_tested("doc/examples/", app_configs, ignore_configs) errors = 0 for app in apps: @@ -170,7 +170,7 @@ config = os.path.join(confpath, config) errors += test_config(app, config, tmpdir, verbose) - if rmtmp or not errors: + if not errors: remove_tmpdir(tmpdir) if errors: -- To view, visit https://gerrit.osmocom.org/c/python/osmo-python-tests/+/36934?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: python/osmo-python-tests Gerrit-Branch: master Gerrit-Change-Id: I469d854cda0a23bbf545953ad2b94666df8e7d04 Gerrit-Change-Number: 36934 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <osmith(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: osmith <osmith(a)sysmocom.de> Gerrit-MessageType: merged