Merge pull request #21060 from mguetschow/make-random-prng-default

makefiles: select default prng _after_ recursive dependency resolution

Makefile.dep

@@ -46,13 +46,6 @@ ifneq (,$(filter libfixmath-unittests,$(USEMODULE)))
   USEMODULE += libfixmath
 endif
 
-ifneq (,$(filter random,$(USEMODULE)))
-  # select default prng if no prng is selected
-  ifeq (,$(filter prng_%,$(USEMODULE)))
-    USEMODULE += prng_musl_lcg
-  endif
-endif
-
 ifneq (,$(filter spiffs,$(USEMODULE)))
   USEPKG += spiffs
   USEMODULE += vfs

makefiles/defaultmodules_deps.inc.mk

@@ -22,6 +22,13 @@ ifneq (,$(filter auto_init_libcose_crypt,$(USEMODULE)))
   USEMODULE += libcose_crypt_init
 endif
 
+ifneq (,$(filter random,$(USEMODULE)))
+  # select default prng if no prng is selected
+  ifeq (,$(filter prng_%,$(USEMODULE)))
+    USEMODULE += prng_musl_lcg
+  endif
+endif
+
 ifneq (,$(filter xtimer,$(USEMODULE)))
   ifeq (,$(filter ztimer_xtimer_compat,$(USEMODULE)))
     USEMODULE += div

tests/build_system/external_module_dirs/README.md

@@ -7,5 +7,5 @@ It demonstrates:
 
  * Adding a module with source code
  * Setting a header include directory
- * Adding dependencies, which are evaluated before other modules dependencies
+ * Adding (transitive) dependencies, which are evaluated before other modules dependencies
  * Registering a module for auto-initialization

tests/build_system/external_module_dirs/external_modules/external_module/Makefile.dep

@@ -1,3 +1 @@
-USEMODULE += random
-# Set a different prng than the default
-USEMODULE += prng_xorshift
+USEMODULE += transitive

tests/build_system/external_module_dirs/external_modules/transitive/Makefile

@@ -0,0 +1 @@
+include $(RIOTBASE)/Makefile.base

tests/build_system/external_module_dirs/external_modules/transitive/Makefile.dep

@@ -0,0 +1,2 @@
+# Set a different prng than the default
+USEMODULE += prng_xorshift