[PATCH] D29185: [lit] Allow llvm's build and test systems to support paths with spaces

ben via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 2 02:38:37 PST 2017


bd1976llvm added inline comments.


================
Comment at: utils/lit/tests/sp aces/spaces-in-paths.py:3
+
+# RUN: %{lit} -j 1 -v check-for-spaces.py
----------------
rnk wrote:
> We could avoid checking in paths with spaces by doing something like:
> ```
> RUN: mkdir "path with space"
> RUN: cp %S/Inputs/check-for-spaces.py "path with space"
> RUN: cd "path with space"
> RUN: %{lit} -j1 -v check-for-spaces.py
> ```
Nice suggestion! Lets see what others think about allowing *just one space* into the repo :)


https://reviews.llvm.org/D29185





More information about the llvm-commits mailing list