<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 17, 2014 at 6:35 AM, Marek Kurdej <span dir="ltr"><<a href="mailto:curdeius@gmail.com" target="_blank">curdeius@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>>! In D5185#7, @thakis wrote:<br>
> This directory contains <a href="https://code.google.com/p/googletest/" target="_blank">https://code.google.com/p/googletest/</a> . I think the way to go about changing it is to get the change merged upstream, and then update this directory to what's in the upstream repo (?)<br>
<br>
</span>I checked the googletest page. The current version does not have this problem, because isascii is not used in 1.7.0.<br>
LLVM uses a slightly modified version 1.6.0, so I see two possibilities:<br>
<br>
  # Update googletest in LLVM to 1.7.0 (However, I do not know the general policy for handling external dependencies in LLVM)<br>
  # Patch current version (That is what I propose, a short-term solution)<br></blockquote></div><br></div><div class="gmail_extra">Upgrading to 1.7 sounds like it leads to a better place long-term than staying on 1.6 and patching it up. Do others on this list have any opinion on this? Does LLVM have existing policies for how to handle third-party dependencies?</div></div>