[Lldb-commits] [PATCH] D132257: [lldb] Create flag to use a custom libcxx in tests

Fri Aug 19 13:30:29 PDT 2022

aprantl added inline comments.

================
Comment at: lldb/packages/Python/lldbsuite/test/dotest.py:338-339

+    if args.custom_libcpp:
+        os.environ['CUSTOM_LIBCPP'] = args.custom_libcpp
+
----------------
JDevlieghere wrote:
> aprantl wrote:
> > JDevlieghere wrote:
> > > Please don't rely on environment variables to pass arguments to the Make invocation. This makes it really tedious to debug make invocations. Instead, pass these explicitly as part of the make invocation from the builders (`packages/Python/lldbsuite/test/builders/`). 
> > That is a very good point. Maybe we should just fix the -E option, which doesn't work anyway due to Makefiles setting CFLAGS_EXTRAS and use that.
> I think a specific flag is fine, we already have something similar for using the "hermetic" libc++ (I left a comment below about that). We should see if we can unify this.
It looks like (apart from making -E not use CFLAGS_EXTRAS but a fresh variable) we should use that mechanism for all the options passed using os.environ().

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132257