fixed #13365 - addons/test/util.py: got rid of non-portable commands (d…

…anmar#7064)
swasti16 · Dec 5, 2024 · 9fbc137 · 9fbc137
1 parent 1a5743c
commit 9fbc137
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/addons/test/misra_test.py b/addons/test/misra_test.py
@@ -116,7 +116,6 @@ def test_rules_suppression(checker, capsys):
 
     for src in test_sources:
         re_suppressed= r"\[%s\:[0-9]+\]" % src
-        dump_remove(src)
         dump_create(src, "--suppressions-list=addons/test/misra/suppressions.txt","--inline-suppr")
         checker.parseDump(src + ".dump")
         captured = capsys.readouterr().err

diff --git a/addons/test/util.py b/addons/test/util.py
@@ -25,13 +25,10 @@ def dump_create(fpath, *argv):
         p.communicate()
         if p.returncode != 0:
             raise OSError("cppcheck returns error code: %d" % p.returncode)
-    with subprocess.Popen(["sync"]) as p:
-        p.communicate()
 
 
 def dump_remove(fpath):
-    with subprocess.Popen(["rm", "-f", fpath + ".dump"]) as p:
-        p.communicate()
+    os.remove(fpath + ".dump")
 
 
 def convert_json_output(raw_json_strings):