[riot-notifications] [RIOT-OS/RIOT] sys/evtimer: introduce ZTIMER_MSEC as timer backend (#13661)

Kaspar Schleiser notifications at github.com
Tue May 19 20:23:08 CEST 2020


> I think @maribu's point is the recursive dependency resolution by including all `Makefile.dep` over and over again until the result hasn't altered during a run. So in this case `PSEUDOMODULES` ends up with containing `xtimer_on_ztimer` several times.
> 
> This does work but takes some time.

Can we quantify this? The recursion stops when there are no changes (additions) to the checked variables. Adding to PSEUDOMODULES doesn't even cause deeper recursion. And if having a module in there multiple times probably won't have any measurable effect on performance. Neither would a `PSEUDOMODULES := $(sort $ÜSEUDOMODULES))`, which would deduplicate it.

You can certainly "fix" this, but IMO there's no issue.

-- 
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/13661#issuecomment-630997096
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200519/b020770d/attachment.htm>


More information about the notifications mailing list