[riot-notifications] [RIOT-OS/RIOT] make: add capability to check config for `test-with-config` (#16795)

Karl Fessel notifications at github.com
Fri Sep 3 17:16:49 CEST 2021

@kfessel commented on this pull request.

>  	$(Q) for t in $(TESTS_WITH_CONFIG); do \
 		$$t || exit 1; \
-	$(Q)test -n "$(strip $(TESTS_WITH_CONFIG))"
+# run the automatic configuration check
+	$(Q)test -z "$(strip $(CHECK_CONFIG))" || \
+		${COLOR_ECHO} -n "${COLOR_RED}"; \
+		for t in $(CHECK_CONFIG) empty; do \
+			test $$t = "empty" && break; \
+			$$t || \
+			(echo "System configuration for" \
+			"tests is not available, check $${t#$${RIOTBASE}/} failed." \

" check tests/gnrc_dhcpv6_client/tests-with-config/check-config.sh failed. "
 i read the check a a verb and it sounded wrong to me but if check and the filename are one nous it sound correct

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210903/c4a3b154/attachment.htm>

More information about the notifications mailing list