<p dir="ltr">You can feel free to change it from single to double quotes if that fixes your issue</p>
<br><div class="gmail_quote"><div dir="ltr">On Fri, Oct 6, 2017, 8:02 PM Bruno Cardoso Lopes <<a href="mailto:bruno.cardoso@gmail.com">bruno.cardoso@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Francis,<br>
<br>
On Wed, Oct 4, 2017 at 10:30 AM, Francis Ricci via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> Author: fjricci<br>
> Date: Wed Oct  4 10:30:28 2017<br>
> New Revision: 314915<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=314915&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=314915&view=rev</a><br>
> Log:<br>
> [test] Fix append_path in the empty case<br>
><br>
> Summary:<br>
> normpath() was being called on an empty string and appended to<br>
> the environment variable in the case where the environment variable<br>
> was unset. This led to ":." being appended to the path, since<br>
> normpath() of an empty string is '.', presumably to represent cwd.<br>
<br>
Thanks for fixing this. However, I'm not quite sure I get part of<br>
this. Looking at parse_flags() in<br>
lib/sanitizer_common/sanitizer_flag_parser.cc, it looks to me like the<br>
separator is a whitespace. os.path.pathsep defaults to ":" on darwin,<br>
as you notice in your comment. I guess was never a issue for<br>
ASAN_OPTIONS because it had never more than one?<br>
<br>
Now I get issues with these changes when I run this in macOS 10.13.<br>
The workaround I found was to actually quote the arg:<br>
<br>
diff --git a/utils/lit/lit/llvm/config.py b/utils/lit/lit/llvm/config.py<br>
index 87851b3cb1a..c381e2fd040 100644<br>
--- a/utils/lit/lit/llvm/config.py<br>
+++ b/utils/lit/lit/llvm/config.py<br>
@@ -82,7 +82,7 @@ class LLVMConfig(object):<br>
             if re.match(r'^x86_64.*-apple', target_triple):<br>
                 if 'address' in sanitizers:<br>
                     self.with_environment(<br>
-                        'ASAN_OPTIONS', 'detect_leaks=1', append_path=True)<br>
+                        'ASAN_OPTIONS', "'detect_leaks=1'", append_path=True)<br>
             if re.match(r'^x86_64.*-linux', target_triple):<br>
                 features.add('x86_64-linux')<br>
             if re.match(r'.*-win32$', target_triple):<br>
<br>
Ideas?<br>
<br>
--<br>
Bruno Cardoso Lopes<br>
<a href="http://www.brunocardoso.cc" rel="noreferrer" target="_blank">http://www.brunocardoso.cc</a><br>
</blockquote></div>