[PATCH] D37838: [lit] Allow lit config files to have a .py extension

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 13 17:22:39 PDT 2017


zturner created this revision.
Herald added a subscriber: mgorny.
Herald added a reviewer: modocache.

Not having a .py extension breaks many python related tools, and is generally kind of obnoxious.

This is a minor quality of life improvement to allow lit config files to have `.py` extensions.  The original filenames are still supported, now you just have the *option* of having a lit config file with a `.py` extension.  I didn't want to get too heavy handed, so I only changed the configs in `llvm/test` and `llvm/test/Unit`


https://reviews.llvm.org/D37838

Files:
  llvm/cmake/modules/AddLLVM.cmake
  llvm/test/CMakeLists.txt
  llvm/test/Unit/lit.cfg
  llvm/test/Unit/lit.cfg.py
  llvm/test/Unit/lit.site.cfg.in
  llvm/test/Unit/lit.site.cfg.py.in
  llvm/test/lit.cfg
  llvm/test/lit.cfg.py
  llvm/test/lit.site.cfg.in
  llvm/test/lit.site.cfg.py.in
  llvm/utils/lit/lit/LitConfig.py
  llvm/utils/lit/lit/discovery.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D37838.115140.patch
Type: text/x-patch
Size: 7433 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170914/f96a219f/attachment.bin>


More information about the llvm-commits mailing list