[riot-notifications] [RIOT-OS/RIOT] sys/evtimer: fix module dependencies (#15792)

Martine Lenders notifications at github.com
Tue Jan 19 11:34:30 CET 2021


Mhhh when I am on this PR's branch I still have `xtimer` in the list of modules when build like described in the testing procedures:

```console
$ git show
commit 8f5d366fe9176a50da844cdc0a779a52e3349684 (HEAD)
Author: Hauke Petersen <devel at haukepetersen.de>
Date:   Tue Jan 19 11:05:23 2021 +0100

    sys/evtimer: fix module dependencies

diff --git a/sys/Makefile.dep b/sys/Makefile.dep
index 669c30ff0d..b13c0d6155 100644
--- a/sys/Makefile.dep
+++ b/sys/Makefile.dep
@@ -1063,7 +1063,11 @@ ifneq (,$(filter ztimer% %ztimer,$(USEMODULE)))
 endif
 
 ifneq (,$(filter evtimer,$(USEMODULE)))
-  USEMODULE += xtimer
+  ifneq (,$(filter evtimer_on_ztimer,$(USEMODULE)))
+    USEMODULE += ztimer_msec
+  else
+    USEMODULE += xtimer
+  endif
 endif
 
 # handle xtimer's deps. Needs to be done *after* ztimer
$ USEMODULE="evtimer_on_ztimer ztimer_usec" make -C tests/evtimer_msg/ -j clean flash info-modules
make: Entering directory '/home/mlenders/Repositories/RIOT-OS/RIOT/tests/evtimer_msg'
Building application "tests_evtimer_msg" for "native" with MCU "native".

auto_init
auto_init_xtimer
auto_init_ztimer
board
core
core_idle_thread
core_init
core_msg
core_panic
cpu
div
evtimer
evtimer_on_ztimer
frac
native_drivers
periph
periph_common
periph_gpio
periph_gpio_linux
periph_init
periph_init_gpio
periph_init_gpio_linux
periph_init_pm
periph_init_timer
periph_init_uart
periph_pm
periph_timer
periph_uart
stdin
stdio_native
sys
test_utils_interactive_sync
xtimer
xtimer_on_ztimer
ztimer
ztimer_auto_init
ztimer_convert
ztimer_convert_frac
ztimer_convert_shift
ztimer_core
ztimer_extend
ztimer_msec
ztimer_now64
ztimer_periph_timer
ztimer_usec
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/boards/native
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/core
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/cpu/native
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/drivers
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/sys
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/boards/native/drivers
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/cpu/native/periph
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/cpu/native/stdio_native
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/drivers/periph_common
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/sys/auto_init
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/sys/div
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/sys/evtimer
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/sys/frac
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/sys/test_utils/interactive_sync
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/sys/xtimer
"make" -C /home/mlenders/Repositories/RIOT-OS/RIOT/sys/ztimer
/usr/bin/ld: /home/mlenders/Repositories/RIOT-OS/RIOT/tests/evtimer_msg/bin/native/cpu/tramp.o: warning: relocation against `_native_saved_eip' in read-only section `.text'
/usr/bin/ld: warning: creating DT_TEXTREL in a PIE
   text	   data	    bss	    dec	    hex	filename
  47374	    732	  60492	 108598	  1a836	/home/mlenders/Repositories/RIOT-OS/RIOT/tests/evtimer_msg/bin/native/tests_evtimer_msg.elf
true
make: Leaving directory '/home/mlenders/Repositories/RIOT-OS/RIOT/tests/evtimer_msg'
```

-- 
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/15792#issuecomment-762754202
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210119/649c79fa/attachment-0001.htm>


More information about the notifications mailing list