jasonmolenda marked 2 inline comments as done.
jasonmolenda added inline comments.
================
Comment at:
lldb/test/API/python_api/target-arch-from-module/TestTargetArchFromModule.py:39
+ dwarfdump_cmd_output = subprocess.check_output(
+ ('/usr/bin/dwarfdump --uuid "%s"' % aout_exe), shell=True
+ ).decode("utf-8")
----------------
bulbazord wrote:
> I know this is already done in a few places but we should really use
> `llvm-dwarfdump` so this can be a more portable test...
Yeah Jonas made that point last week about another test I was getting the uuid
from a file. I don't know how to refer to something in the build products
directory in an API test like this, is that what you meant? It's not installed
on a standard macOS system in any path.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157659/new/
https://reviews.llvm.org/D157659
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
