<div>Hi Daniel,</div>
<div> </div>
<div>Thanks.</div>
<div> </div>
<div>> By the way, I still don't have all my local patches in for testing on<br>Windows (there are basically 3, one to add count + not to<br>test/Scripts, one to add my local search headers, and one to hack<br>around stdint.h), but the current number of test failures is ~60,<br>
which isn't too bad. A lot of the remaining ones are STL iterator<br>pickyness which should be easy to eliminate if someone sits down to<br>work through them.<br></div>
<div>I still have the local patches you gave me before, which I was using, somewhat successfully, though the three of use trying the tests saw some different results.  But that was a couple months ago, with me being pulled onto other projects and vacation, so I'll see about running them again, if I can figure out how I was running them before.  I'll start looking at those test failures, hoping it's not too far over my head.  I do look forward to having the full solution for running the tests.</div>

<div> </div>
<div>> Also, what happened with your patch to add MSVC search paths in a more<br>principled fashion? In my fuzzy memory I thought it had gone in, but I<br>didn't actually see it in the source.<br></div>
<div>Regarding the include path patch, it was kind of a hack job, mainly to facilitate our development.  I think you or someone raised some objection about one part where when both vc80 and c90 are present (both the VS80COMNTOOLS and VS90COMNTOOLS environment variables are set) and I used the one Clang was built with.  I kind of left it at that.</div>

<div> </div>
<div>How do you think I should handle this case?  Just use whichever is the later VS release?<br></div>
<div>I've enclosed a refreshed patch, in case you want to see it again.  It still has some hard-coded paths like before, which is also why it's kind of hackish.</div>
<div> </div>
<div>-John</div>
<div> </div>
<div class="gmail_quote">On Wed, Sep 23, 2009 at 12:34 AM, Daniel Dunbar <span dir="ltr"><<a href="mailto:daniel@zuster.org">daniel@zuster.org</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">Hi John,<br><br>Thanks for the refactoring, it looks good to me, I applied as r82621.<br>
<div class="im"><br>On Tue, Sep 22, 2009 at 11:21 PM, John Thompson<br><<a href="mailto:john.thompson.jtsoftware@gmail.com">john.thompson.jtsoftware@gmail.com</a>> wrote:<br>> Hi Daniel,<br>> Sorry for the duplicate email, I keep forgetting to pay attention to the<br>
> return address.<br>> I've redone the patch without the Triple changes.  The current triple<br>> mechanism seems a bit out-of-date, as the correct triple for 64-bit Windows<br>> appears to be "x86_64-pc-win32", but we can leave that for whoever owns it.<br>
<br></div>Ok.<br>
<div class="im"><br>> At some point, if Clang wants to be it's own environment on Windows, we'll<br>> probably have to revise this a bit to be able to do something other than<br>> default to Visual Studio for the "win32" OS-type, but this is probably good<br>
> enough for now.<br><br></div>Yeah, we will probably want to go a long time before we care about<br>this. Adoption means being compatible with the existing environment,<br>as painful as it may be.<br>
<div class="im"><br>> Thanks for dealing with this for me.<br>><br>> -John<br>><br>> P.S.  I added an empty define for __declspec.  Is this all that's needed?<br><br></div>Dunno...<br><br>By the way, I still don't have all my local patches in for testing on<br>
Windows (there are basically 3, one to add count + not to<br>test/Scripts, one to add my local search headers, and one to hack<br>around stdint.h), but the current number of test failures is ~60,<br>which isn't too bad. A lot of the remaining ones are STL iterator<br>
pickyness which should be easy to eliminate if someone sits down to<br>work through them.<br><br>Also, what happened with your patch to add MSVC search paths in a more<br>principled fashion? In my fuzzy memory I thought it had gone in, but I<br>
didn't actually see it in the source.<br><font color="#888888"><br> - Daniel<br></font></blockquote></div><br><br clear="all">
<div></div><br>-- <br>John Thompson<br><a href="mailto:John.Thompson.JTSoftware@gmail.com">John.Thompson.JTSoftware@gmail.com</a><br><br>