Skip to content

Commit 3654d84

Browse files
committed
Merge bitcoin/bitcoin#28245: doc: use llvm-config for bitcoin-tidy example
d82bb90 doc: use llvm-config for bitcoin-tidy example (fanquake) Pull request description: An LLVM installation will have `llvm-config` available to query for info. Ask it for the `--cmakedir`, and use that in our bitcoin-tidy example, rather than listing multiple different (potential) paths per distro/OS etc. ACKs for top commit: theuni: ACK d82bb90. jonatack: ACK d82bb90 TheCharlatan: Nice, Re-ACK d82bb90 Tree-SHA512: e07e979231f8f000deafce0751bed4b73ff0eff995bec49e90f579c9051cf5859dac5e49554b8219d33b00c81192db979eed98fee1c643a9205ea8babfce2c5d
2 parents b2ec032 + d82bb90 commit 3654d84

File tree

1 file changed

+5
-2
lines changed
  • contrib/devtools/bitcoin-tidy

1 file changed

+5
-2
lines changed

contrib/devtools/bitcoin-tidy/README

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@
33
Example Usage:
44

55
```bash
6-
cmake -S . -B build -DLLVM_DIR=/path/to/lib/cmake/llvm -DCMAKE_BUILD_TYPE=Release
7-
make -C build -j$(nproc)
6+
cmake -S . -B build -DLLVM_DIR=$(llvm-config --cmakedir) -DCMAKE_BUILD_TYPE=Release
7+
8+
cmake --build build -j$(nproc)
9+
10+
cmake --build build --target bitcoin-tidy-tests -j$(nproc)
811
```

0 commit comments

Comments
 (0)