[PATCH] D64135: [lit] Parse command-line options from LIT_OPTS

[Joel E. Denny via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL365313: [lit] Parse command-line options from LIT_OPTS (authored by jdenny, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D64135?vs=208055&id=208369#toc

Repository:
  rL LLVM

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

https://reviews.llvm.org/D64135

Files:
  llvm/trunk/docs/CommandGuide/lit.rst
  llvm/trunk/utils/lit/lit/main.py
  llvm/trunk/utils/lit/tests/Inputs/lit-opts/lit.cfg
  llvm/trunk/utils/lit/tests/Inputs/lit-opts/test.txt
  llvm/trunk/utils/lit/tests/lit-opts.py


Index: llvm/trunk/utils/lit/lit/main.py
===================================================================
--- llvm/trunk/utils/lit/lit/main.py
+++ llvm/trunk/utils/lit/lit/main.py
@@ -11,6 +11,7 @@
 import platform
 import random
 import re
+import shlex
 import sys
 import time
 import argparse
@@ -322,7 +323,8 @@
                       help="Show all discovered tests",
                       action="store_true", default=False)
 
-    opts = parser.parse_args()
+    opts = parser.parse_args(sys.argv[1:] +
+                             shlex.split(os.environ.get("LIT_OPTS", "")))
     args = opts.test_paths
 
     if opts.show_version:
Index: llvm/trunk/utils/lit/tests/lit-opts.py
===================================================================
--- llvm/trunk/utils/lit/tests/lit-opts.py
+++ llvm/trunk/utils/lit/tests/lit-opts.py
@@ -0,0 +1,33 @@
+# Check cases where LIT_OPTS has no effect.
+#
+# RUN:                 %{lit} -j 1 -s %{inputs}/lit-opts | FileCheck %s
+# RUN: env LIT_OPTS=   %{lit} -j 1 -s %{inputs}/lit-opts | FileCheck %s
+# RUN: env LIT_OPTS=-s %{lit} -j 1 -s %{inputs}/lit-opts | FileCheck %s
+
+# Check that LIT_OPTS can override command-line options.
+#
+# RUN: env LIT_OPTS=-a \
+# RUN: %{lit} -j 1 -s %{inputs}/lit-opts \
+# RUN: | FileCheck -check-prefix=SHOW-ALL -DVAR= %s
+
+# Check that LIT_OPTS understands multiple options with arbitrary spacing.
+#
+# RUN: env LIT_OPTS='-a -v  -Dvar=foobar' \
+# RUN: %{lit} -j 1 -s %{inputs}/lit-opts \
+# RUN: | FileCheck -check-prefix=SHOW-ALL -DVAR=foobar %s
+
+# Check that LIT_OPTS parses shell-like quotes and escapes.
+#
+# RUN: env LIT_OPTS='-a   -v -Dvar="foo bar"\ baz' \
+# RUN: %{lit} -j 1 -s %{inputs}/lit-opts \
+# RUN: | FileCheck -check-prefix=SHOW-ALL -DVAR="foo bar baz" %s
+
+# CHECK:      Testing: 1 tests
+# CHECK-NOT:  PASS
+# CHECK:      Expected Passes : 1
+
+# SHOW-ALL:     Testing: 1 tests
+# SHOW-ALL:     PASS: lit-opts :: test.txt (1 of 1)
+# SHOW-ALL:     {{^}}[[VAR]]
+# SHOW-ALL-NOT: PASS
+# SHOW-ALL:     Expected Passes : 1
Index: llvm/trunk/utils/lit/tests/Inputs/lit-opts/test.txt
===================================================================
--- llvm/trunk/utils/lit/tests/Inputs/lit-opts/test.txt
+++ llvm/trunk/utils/lit/tests/Inputs/lit-opts/test.txt
@@ -0,0 +1 @@
+# RUN: echo %var
Index: llvm/trunk/utils/lit/tests/Inputs/lit-opts/lit.cfg
===================================================================
--- llvm/trunk/utils/lit/tests/Inputs/lit-opts/lit.cfg
+++ llvm/trunk/utils/lit/tests/Inputs/lit-opts/lit.cfg
@@ -0,0 +1,7 @@
+import lit.formats
+config.name = 'lit-opts'
+config.suffixes = ['.txt']
+config.test_format = lit.formats.ShTest()
+config.test_source_root = None
+config.test_exec_root = None
+config.substitutions.append(('%var', lit_config.params.get('var', '')))
Index: llvm/trunk/docs/CommandGuide/lit.rst
===================================================================
--- llvm/trunk/docs/CommandGuide/lit.rst
+++ llvm/trunk/docs/CommandGuide/lit.rst
@@ -38,6 +38,11 @@
 subset of the options specified on the command line, see
 :ref:`selection-options` for more information.
 
+:program:`lit` parses options from the environment variable ``LIT_OPTS`` after
+parsing options from the command line.  ``LIT_OPTS`` is primarily useful for
+supplementing or overriding the command-line options supplied to :program:`lit`
+by ``check`` targets defined by a project's build system.
+
 Users interested in the :program:`lit` architecture or designing a
 :program:`lit` testing implementation should see :ref:`lit-infrastructure`.
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64135.208369.patch
Type: text/x-patch
Size: 3585 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190708/d07e61e5/attachment.bin>