[riot-notifications] [RIOT-OS/RIOT] build system: Rework EXTERNAL_MODULE_DIRS (#16104)

benpicco notifications at github.com
Tue May 4 12:25:28 CEST 2021


@benpicco approved this pull request.

works like a charm!

All I had to do was

```patch
--- a/modules/external_module_dirs.mk
+++ b/modules/external_module_dirs.mk
@@ -1,4 +1 @@
-MODULEDIR ?= $(abspath $(dir $(lastword $(MAKEFILE_LIST))))
-
-KNOWN_EXTERNAL_MODULES = $(foreach md, $(dir $(wildcard $(MODULEDIR)/*/.)), $(shell basename $(md)))
-EXTERNAL_MODULE_DIRS += $(addprefix $(MODULEDIR)/, $(filter $(KNOWN_EXTERNAL_MODULES), $(USEMODULE)))
+EXTERNAL_MODULE_DIRS := $(dir $(lastword $(MAKEFILE_LIST)))
```



-- 
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/16104#pullrequestreview-651089054
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210504/b09649fd/attachment.htm>


More information about the notifications mailing list