<div dir="ltr">Nope...<div><br></div><div> - Daniel</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 4, 2015 at 10:16 AM, Steven Wu <span dir="ltr"><<a href="mailto:stevenwu@apple.com" target="_blank">stevenwu@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">cc Daniel. Do you remember the reason why you add SDKROOT to lit for clang?<span class="HOEnZb"><font color="#888888"><div><br></div><div>Steven</div></font></span><div><div class="h5"><div><br><div><blockquote type="cite"><div>On Feb 3, 2015, at 2:10 PM, Steven Wu <<a href="mailto:stevenwu@apple.com" target="_blank">stevenwu@apple.com</a>> wrote:</div><br><div><div style="word-wrap:break-word">Hi Reid<div><br></div><div>Thanks for the suggestion. I am aware of the setting in lit.cfg but I assume some of the environment variables I want to remove from the driver tests might be useful in some other cases. </div><div>For example, SDKROOT is setup by lit.cfg. Although I am not aware of any clang tests will fail on mac os if this environment is not set (maybe compiler_rt really need it), setting that environment to the wrong path will definitely fail many driver tests. I am just being cautious by removing them only from driver tests, but I can try remove them from all clang tests and see what will happen.</div><div><br></div><div>Steven</div><div> <br><div><blockquote type="cite"><div>On Feb 3, 2015, at 1:55 PM, Reid Kleckner <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> wrote:</div><br><div><div dir="ltr">We already have this mechanism in test/lit.cfg. Feel free to add some more variables.</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 3, 2015 at 1:17 PM, Steven Wu <span dir="ltr"><<a href="mailto:stevenwu@apple.com" target="_blank">stevenwu@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping. This is intended to avoid driver tests gets broken by the unexpected environment variable. They can be accidentally set by user or bots and report false positive in driver tests.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D7135" target="_blank">http://reviews.llvm.org/D7135</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>
</div></blockquote></div><br></div></div>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br></div></blockquote></div><br></div></div></div></div></blockquote></div><br></div>