From 927d2cd67f9fdd78957bca52f5545474dc6ce032 Mon Sep 17 00:00:00 2001
From: Marian Buschsieweke <marian.buschsieweke@ovgu.de>
Date: Sun, 21 May 2023 10:25:02 +0200
Subject: [PATCH] pkg/fff: disable LLVM in CI, error message on old clang

pkg/fff compiles fine with clang 16.0.0 and higher, but the CI currently
uses version 14.x.y which is incompatible. This disables LLVM in the CI
builds and prints a friendly error message when `TOOLCHAIN=llvm` is
used and clang is older than 16.0.0.
---
 pkg/fff/Makefile.include | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/pkg/fff/Makefile.include b/pkg/fff/Makefile.include
index ef26e83006..698fcfd20d 100644
--- a/pkg/fff/Makefile.include
+++ b/pkg/fff/Makefile.include
@@ -6,3 +6,14 @@ PSEUDOMODULES += fff
 
 # Tests don't need pedantic. Pedantic throws errors in variadic macros when compiling for C++
 CXXEXFLAGS += -Wno-pedantic
+
+ifeq (1,$(RIOT_CI_BUILD))
+  # TODO: remove this when the build containers clang is updated to version 16
+  TOOLCHAINS_BLACKLIST += llvm
+else
+  ifeq (llvm,$(TOOLCHAIN))
+    ifneq (1,$(call version_is_greater_or_equal,$(LLVM_VERSION),16))
+      $(error package fff requires clang 16.0.0 or newer. Use GCC or update clang.)
+    endif
+  endif
+endif