[Lldb-commits] [PATCH] D141042: [lldb] Allow configuring on Windows with python interpreter within a junction

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.

Looks reasonable to me.

@lawrence_danna do you happen to remember why you wrote:

  exe = os.path.join(os.path.realpath(os.path.dirname(exe)), os.readlink(exe))

instead of

  exe = os.path.realpath(exe)

which seems like it should be equivalent? Aand you're doing a `realpath` call anyway...


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141042/new/

https://reviews.llvm.org/D141042