[riot-notifications] [RIOT-OS/RIOT] ztimer: relocate PSEUDOMULDES definitions (#14123)

Marian Buschsieweke notifications at github.com
Mon May 25 13:16:06 CEST 2020


I wonder if it wouldn't be simpler to move the include of `pseudomodules.inc.mk` from here:

https://github.com/RIOT-OS/RIOT/blob/4adb8ab402e2a523c984003375a6a543a0b6c5b3/Makefile.include#L367-L369

below here:

https://github.com/RIOT-OS/RIOT/blob/4adb8ab402e2a523c984003375a6a543a0b6c5b3/Makefile.include#L396-L397

The `PSEUDOMODULES` should not be relevant for the dependency resolution anyway, so we could just process them after and do within `pseudomodules.inc.mk` stuff like:

``` Makefile
ifneq (,$(filter ztimer_xtimer_compat,$(USEMODULE))
  # Don't link in xtimer when ztimer_xtimer_compat is used
  PSEUDOMOUDLES += xtimer
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/14123#issuecomment-633521202
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200525/0b03d9ef/attachment.htm>


More information about the notifications mailing list