<div dir="ltr">Thanks, I'll commit this later today then.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Dec 4, 2015 at 10:25 AM Todd Fiala <<a href="mailto:todd.fiala@gmail.com">todd.fiala@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tfiala accepted this revision.<br>
tfiala added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
Hey Zachary,<br>
<br>
As best as I can tell, we don't need to do anything here, at least not on OS X El Capitan, because six is already included in the OS X Python distribution.<br>
<br>
I was scratching my head for a while to figure out why, but that seems to be it.<br>
<br>
I'd be okay with leaving this as is without any Xcode change.  I will be cleaning up the "finish" step of the python script handling, much like I did for the front end a few weeks ago.  When I do that, Xcode will switch to the finisher, and will pick up the "lldb official" in-repo version of six we're using.  I'd rather not create the extra work of addressing "which six" at this point given that other pending work.<br>
<br>
All the tests run on a clean build after with no new errors.  (OS X right now seems to have one error and one failure on a clean public system at this point, but that is not a result of this change).  I also exercised the in-lldb script handling and that seemed to be working.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D15209" rel="noreferrer" target="_blank">http://reviews.llvm.org/D15209</a><br>
<br>
<br>
<br>
</blockquote></div>