diff options
author | André Schröder <andre.schroedr@gmail.com> | 2021-08-02 22:43:26 +0000 |
---|---|---|
committer | André Schröder <andre.schroedr@gmail.com> | 2021-08-02 22:43:26 +0000 |
commit | 587f595f46f7d99d5f7beba5d3fb31dfaabc2d5c (patch) | |
tree | a43c483fe9421c8b96c35e576922805009c3315b /test | |
parent | fix for llvm 13 (diff) | |
download | cgrep-587f595f46f7d99d5f7beba5d3fb31dfaabc2d5c.tar.gz cgrep-587f595f46f7d99d5f7beba5d3fb31dfaabc2d5c.zip |
[patch] Use a reasonable fallback for LLVM_CONF
In #19 I wrote that I couldn't compile the code via
$ cmake -B build -S cgrep/ -DUSE_MONOLITH_LIBTOOLING=ON -DCMAKE_CXX_COMPILER=clang++
$ cmake --build build
This is because I didn't set LLVM_CONF and it falls back to
llvm-config-10. There are two problems with this:
First, CMake didn't actually check if LLVM_CONF existed. I now check it
via the REQUIRED argument in find_program.
Second, my OS (Arch Linux) doesn't name the binary like llvm-config-10
but it's called llvm-config, so the fallback never works on Arch. Thus,
I added a non-complete but somewhat reasonable list of possible names.
Diffstat (limited to 'test')
0 files changed, 0 insertions, 0 deletions