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

Francisco notifications at github.com
Tue May 4 10:36:44 CEST 2021


> Some testing where I removed the `external_modules` directory in `tests/external_module_dirs` and put it outside of tree and exported the location:
> compile test OK
> compile test in docker OK
> compile test with setup matching previous handling FAIL
> 
> So the thing that remains IMO is how to notify external users of the change, since the variable name has not changed it's hard to add a warning. Too bad the current name is quite good already.... I would suggest adding a warning if `EXTERNAL_MODULE_DIRS` is set for a release or two, what do you think @maribu?

```
ifneq (,$(EXTERNAL_BOARD_DIRS))
  $(info Warning! EXTERNAL_BOARD_DIRS is a serach folder since 2021.07-branch, see https://doc.riot-os.org/creating-modules.html)
endif
```

-- 
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#issuecomment-831775129
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210504/002eab27/attachment.htm>


More information about the notifications mailing list