[Lldb-commits] [PATCH] D85258: [test] Use realpath consistently for test root file paths.

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Aug 4 18:49:08 PDT 2020


JDevlieghere added inline comments.


================
Comment at: lldb/packages/Python/lldbsuite/test/lldbtest.py:499
         # /abs/path/to/packages/group/subdir/mytest.py -> group/subdir
-        rel_prefix = test_file[len(os.environ["LLDB_TEST_SRC"]) + 1:]
-        return os.path.dirname(rel_prefix)
+        lldb_test_src = os.environ["LLDB_TEST_SRC"]
+        if not test_file.startswith(lldb_test_src):
----------------
While you are here... can you change this to pass the source directory trough the `configuration`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85258/new/

https://reviews.llvm.org/D85258



More information about the lldb-commits mailing list