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

MrKevinWeiss notifications at github.com
Mon Jun 28 10:28:30 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")

Turns out it was pretty easy.  I tested by changing helloworld to cpp.  I also exposed the CPP and cppstdlib modules as it does make sense to select them.

-- 
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_r659582830
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210628/9c95537b/attachment-0001.htm>


More information about the notifications mailing list