tmp

.github/workflows/build-all.yaml

@@ -128,9 +128,7 @@ jobs:
 
       - name: install autotools
         run: |
-          which clang
           brew install autoconf automake libtool
-          which clang
 
       - uses: actions/checkout@v4
 

scripts/dev.py

@@ -589,10 +589,24 @@ def build_fribidi(release=False):
 
 
             # build wasm
+            clang = 'clang'
+
+            #NOTE(martin): this is an extremely stupid workaround to play well with github CI runners, which
+            # have llvm clang only accessible through $(brew --prefix llvm@15), whereas locally we could want to
+            # use another version.
+            # TODO: we should probably pass a flag to inform the script it's running in CI. This would avoid picking
+            # llvm 15 when a later version is available locally?
+            if platform.system() == "Darwin":
+                try:
+                    brew_llvm = subprocess.check_output(["brew", "--prefix", "llvm@15", "--installed"], stderr=subprocess.DEVNULL).decode().strip()
+                except subprocess.CalledProcessError:
+                    brew_llvm = subprocess.check_output(["brew", "--prefix", "llvm", "--installed"]).decode().strip()
+                clang = os.path.join(brew_llvm, 'bin', 'clang')
+
             source_files = glob.glob("lib/*.c");
 
             subprocess.run([
-                "clang",
+                clang,
                 "--target=wasm32",
                 "--no-standard-libraries",
                 "-mbulk-memory",