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

Francisco notifications at github.com
Mon May 3 08:13:42 CEST 2021


@fjmolinas commented on this pull request.



> +# Locate used modules in $(EXTERNAL_MODULE_DIRS).
+EXTERNAL_MODULE_PATHS := $(sort $(foreach dir,$(EXTERNAL_MODULE_DIRS),\
+  $(foreach mod,$(USEMODULE),$(dir $(wildcard $(dir)/$(mod)/Makefile)))))
+

Re-looking at the code, and comment/question in https://github.com/RIOT-OS/RIOT/pull/16104#issuecomment-806163303, this would mean all `EXTERNAL_MODULE_DIRS` would need to be absolute paths no?



-- 
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-650102498
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210502/ff64b6bd/attachment.htm>


More information about the notifications mailing list