[PATCH] D50915: [LNT] In command-line tool expand tilde when resolving commands.

Volodymyr Sapsai via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 17 12:27:16 PDT 2018


vsapsai created this revision.
vsapsai added reviewers: cmatthews, kristof.beyls.
Herald added a subscriber: dexonsmith.

The use case it addresses is being able to provide extra tools like

  --use-lit=~/llvm/build/bin/llvm-lit

Currently lnt fails even if the file is present and executable. Error looks like

> lnt error: LIT tool not found (looked for ~/llvm/build/bin/llvm-lit)


Repository:
  rL LLVM

https://reviews.llvm.org/D50915

Files:
  lnt/testing/util/commands.py


Index: lnt/testing/util/commands.py
===================================================================
--- lnt/testing/util/commands.py
+++ lnt/testing/util/commands.py
@@ -114,6 +114,8 @@
     executable.
 
     """
+    name = os.path.expanduser(name)
+
     # If the given name exists (or is a path), make it absolute.
     if os.path.exists(name):
         return os.path.abspath(name)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50915.161296.patch
Type: text/x-patch
Size: 391 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180817/cfcafd61/attachment.bin>


More information about the llvm-commits mailing list