From 5027cfb6189879ddd0d5cdaca9280235837ce9a0 Mon Sep 17 00:00:00 2001 From: Laytan Laats Date: Sun, 26 May 2024 20:01:57 +0200 Subject: revert `$(llvm-config --bindir)/clang++` to get the cpp compiler A few reports of either people not having this clang++ or having it but it not finding system headers. On top of that, the reason we added this in the first place was a bug that surfaced on clang-18 which we've since fixed. Order will now be clang++ from path, then the llvm bindir clang++, then an error. This can all still be overwritten with `CXX=blah make` like before. --- build_odin.sh | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/build_odin.sh b/build_odin.sh index c27ad119d..ec65bb49d 100755 --- a/build_odin.sh +++ b/build_odin.sh @@ -45,7 +45,13 @@ if [ -z "$LLVM_CONFIG" ]; then fi fi -: ${CXX=$($LLVM_CONFIG --bindir)/clang++} +if [ -x "$(which clang++)" ]; then + : ${CXX="clang++"} +elif [ -x "$($LLVM_CONFIG --bindir)/clang++" ]; then + : ${CXX=$($LLVM_CONFIG --bindir)/clang++} +else + error "No clang++ command found. Set CXX to proceed." +fi LLVM_VERSION="$($LLVM_CONFIG --version)" LLVM_VERSION_MAJOR="$(echo $LLVM_VERSION | awk -F. '{print $1}')" -- cgit v1.2.3