Jason once said that the behavior of Resolve used to be like that of realpath(), which does resolve links.  So there may be some precedent to changing it.  <br><br>The most sensible way to me would be to add llvm::sys::fs::real_path().  I'm not sure if ours used to just call straight through to realpath () though or if it had extra logic, so you'd want to check that.  Also windows doesn't have realpath(), so you'd need to implement it separately there (use CreateFile followed by GetFinalPathNameByHandle to resolve the symlink, plus whatever other logic is baked into realpath())<br><div class="gmail_quote">On Mon, Feb 23, 2015 at 8:04 AM Pavel Labath <<a href="mailto:labath@google.com">labath@google.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Thanks, that is exactly the function i was looking for.<div><br></div><div>However, I would still like to fix the inconsistency between the implementation and the comment of FileSpec::Resolve. Perhaps I should just rephrase the comment, since everyone seems to be happy with the current implementation?</div></div><div dir="ltr"><div><br></div><div>pl</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 23 February 2015 at 15:55, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Please check llvm::sys::fs::equivalent () and see if that does what you want.  If it doesn't, think about whether your change could go there instead of LLDB<br><div class="gmail_quote"><div><div>On Mon, Feb 23, 2015 at 6:30 AM Pavel Labath <<a href="mailto:labath@google.com" target="_blank">labath@google.com</a>> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">Greetings,<div><br></div><div>I need to determine whether two filenames refer to the same file. The description on FileSpec::Resolve reads "Resolves user name and links in path, and overwrites the input</div><div>argument with the resolved path.", which would seem to indicate that it already does symlink resolution. However, this is not the case.</div><div><br></div><div>My question is what is correct. Is it the comment (and then the function should be extended to do symlink resolution), or the implementation (and then we should change the comment to reflect that)?</div><div><br></div><div>I volunteer to do the work, but I am unsure what to do. Does anyone know what is the desired behavior?</div><div><br></div><div>regards,</div><div>pl</div></div></div></div>
______________________________<u></u>_________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@cs.uiuc.edu" target="_blank">lldb-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/lldb-dev</a><br>
</blockquote></div>
</blockquote></div><br></div>
</blockquote></div>