[Lldb-commits] [PATCH] D63790: [dotest] Add the ability to set environment variables for the inferior.

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere marked 2 inline comments as done.
JDevlieghere added inline comments.


================
Comment at: lldb/packages/Python/lldbsuite/test/lldbtest.py:1861
+        if lldbtest_config.inferior_env:
+            self.runCmd('env {}'.format(lldbtest_config.inferior_env))
+
----------------
amccarth wrote:
> `env` or `settings set target.env-vars`?  On Windows, where spaces in paths are common, this works:
> 
>     (lldb) settings set target.env-vars "MY_PATH=C:\Program Files\stuff"
> 
> but this doesn't:
> 
>     (lldb) env "MY_PATH=C:\Program Files\stuff"
> 
> 
I've updated the diff, but this sounds more like a bug in the alias?


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

https://reviews.llvm.org/D63790