[riot-notifications] [RIOT-OS/RIOT] cpu/stm32: Add Kconfig dependency modeling (#16483)

MrKevinWeiss notifications at github.com
Fri Jun 25 11:12:07 CEST 2021


@MrKevinWeiss commented on this pull request.



> @@ -67,9 +67,11 @@ ifeq ($(strip $(ASSMSRC))$(NO_AUTO_SRC),)
   ASSMSRC := $(wildcard *.S)
 endif
 
-ifneq (,$(SRCXX))
-  ifeq (,$(filter cpp,$(FEATURES_USED)))
-    $(error Found C++ source, but feature "cpp" is not used. Add "FEATURES_REQUIRED += cpp")
+ifneq (1,$(TEST_KCONFIG))
+  ifneq (,$(SRCXX))
+    ifeq (,$(filter cpp,$(FEATURES_USED)))
+      $(error Found C++ source, but feature "cpp" is not used. Add "FEATURES_REQUIRED += cpp")

Hmmm true, can kconfig be aware of this? or can we apply a check after kconfig resolved the features?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/16483#discussion_r658616632
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210625/9334b5a1/attachment.htm>


More information about the notifications mailing list