<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Feb 11, 2015, at 5:12 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="">chandlerc@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Wed, Feb 11, 2015 at 1:30 AM, Mehdi Amini <span dir="ltr" class=""><<a href="mailto:mehdi.amini@apple.com" target="_blank" class="">mehdi.amini@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> On Feb 11, 2015, at 1:28 AM, Ilia K <<a href="mailto:ki.stfu@gmail.com" class="">ki.stfu@gmail.com</a>> wrote:<br class="">
><br class="">
> In <a href="http://reviews.llvm.org/D7515#121843" target="_blank" class="">http://reviews.llvm.org/D7515#121843</a>, @joker.eph wrote:<br class="">
><br class="">
>> FWIW, it seems to me that Python is doing it this way: <a href="https://github.com/enthought/Python-2.7.3/blob/master/Lib/posixpath.py" target="_blank" class="">https://github.com/enthought/Python-2.7.3/blob/master/Lib/posixpath.py</a><br class="">
><br class="">
><br class="">
> As you see my fix does the same thing.<br class="">
><br class="">
> Are you interested in this patch? If so I can fix your remarks<br class="">
<br class="">
<br class="">
</span>I am completely neutral. It’s Chandler’s call...<br class="">
<br class="">
We'll see if Python has a strong influence on him :)</blockquote></div><br class="">FWIW, I'd rather not add the complexity. I'm not really swayed that this is important enough for us to add complex logic to handle the fallback case.</div><div class="gmail_extra"><br class=""></div><div class="gmail_extra">What would convince me we should add it would be existing and hard to change settings where HOME was removed from the environment, and thus users who were negatively impacted by us not providing a fallback. Currently, it seems like any failure to find the home directory will likely just expose a flawed runtime environment that needs to be fixed. But if there are users that need LLVM tools to work without the HOME variable *and* who need to access the home directory, then that would motivate adding this support.</div></div></div></blockquote><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><br class=""></div></div></div></blockquote><br class=""></div><div>Completely makes sense to me.</div><div><br class=""></div><div>Mehdi</div><div><br class=""></div><br class=""></body></html>