osmith submitted this change.

View Change


Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified fixeria: Looks good to me, but someone else must approve
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(-)

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 change 36934. To unsubscribe, or for help writing mail filters, visit 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@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-MessageType: merged