It should be, but I haven’t tested any of the non Windows paths at all yet, so while that seems like a logical fix, there’s probably something else that would break, like not having a clang-cl environment.  So the thinking was to just disable them until i can implement and test support for other platforms <br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 5, 2018 at 7:35 AM Pavel Labath <<a href="mailto:pavel@labath.sk">pavel@labath.sk</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 05/12/2018 16:03, Zachary Turner wrote:<br>
> If you can commit the same fix I did for the other two that would be <br>
> great, otherwise I’ll get to it when i get into the office in ~2 hours<br>
<br>
<br>
The thing is, I'm not sure if that's the right fix here. I mean, this <br>
script should be able to compile with clang-cl on linux, right?<br>
<br>
The easiest way to fix this would be to check env is not None before <br>
printing it out, but it wasn't clear to me whether you intended for it <br>
to be (possibly) None at this point in the code, so I figured its best <br>
to let you figure that out.<br>
</blockquote></div>