[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