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

Martine Lenders notifications at github.com
Fri Sep 3 16:35:48 CEST 2021


@miri64 commented on this pull request.



> @@ -40,17 +40,34 @@ test-as-root/available:
 	$(Q)test -n "$(strip $(TESTS_AS_ROOT))"
 
 # Tests that require specific configuration
-.PHONY: test-with-config test-with-config/available
-TESTS_WITH_CONFIG ?= $(foreach file,$(wildcard $(APPDIR)/tests-with-config/*[^~]),\
-                        $(shell test -f $(file) -a -x $(file) && echo $(file)))
+.PHONY: test-with-config test-with-config/available test-with-config/check-config
+TESTS_WITH_CONFIG ?= $(filter-out $(APPDIR)/tests-with-config/check-config%,\

That is not correct however ;-)

-- 
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/16795#discussion_r701940883
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210903/b63e0c36/attachment-0001.htm>


More information about the notifications mailing list