<div dir="ltr">On Thu, Aug 8, 2013 at 8:45 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">> Hm, the problem is that those tests really should test whether PWD is taken<br>
> into account (at least clang-check-pwd).<br>
><br>
> The real fix for those tests is now to create a symlink, cd into the symlink<br>
> directory, set PWD to the symlink path, and assert on the error messages<br>
> that the symlinked path is taken, as that would be the breaking behavior if<br>
> PWD was not taken.<br>
><br>
> Could you please add a comment to the tests that explain this?<br>
> // FIXME: PWD cannot be set to a different path any more; adapt the test to<br>
> build a symlink and verify<br>
> // correct handling of PWD by asserting that clang's error messages contain<br>
> the symlink path instead of<br>
> // the underlying path.<br>
<br>
</div>It is easier to just implement that :-) It even found a missing return<br>
in the llvm patch :-)<br></blockquote><div><br></div><div>Hah, completely missed that :) Awesome!</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im"><br>
> As a side note: if you put changes affecting Tooling through phabricator,<br>
> you'll get my eternal gratitude ;)<br>
><br>
<br>
</div><a href="http://llvm-reviews.chandlerc.com/D1335" target="_blank">http://llvm-reviews.chandlerc.com/D1335</a><br>
<br>
and<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1336" target="_blank">http://llvm-reviews.chandlerc.com/D1336</a><br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div></div>