Merge #19182 #19184

19182: pkg/libfixmath: bump version r=maribu a=benpicco



19184: pkg/tinydtls: bump version r=maribu a=benpicco



Co-authored-by: Benjamin Valentin <benjamin.valentin@ml-pa.com>

core/lib/init.c

@@ -60,7 +60,12 @@ static void *main_trampoline(void *arg)
         LOG_INFO(CONFIG_BOOT_MSG_STRING "\n");
     }
 
-    main();
+    int res = main();
+
+    if (IS_USED(MODULE_TEST_UTILS_MAIN_EXIT_CB)) {
+        void test_utils_main_exit_cb(int res);
+        test_utils_main_exit_cb(res);
+    }
 
 #ifdef MODULE_TEST_UTILS_PRINT_STACK_USAGE
     void print_stack_usage_metric(const char *name, void *stack, unsigned max_size);

makefiles/pseudomodules.inc.mk

@@ -581,6 +581,7 @@ PSEUDOMODULES += crypto_aes_unroll
 
 # declare shell version of test_utils_interactive_sync
 PSEUDOMODULES += test_utils_interactive_sync_shell
+PSEUDOMODULES += test_utils_main_exit_cb
 
 # All auto_init modules are pseudomodules
 PSEUDOMODULES += auto_init_%

pkg/libfixmath/Makefile

@@ -1,5 +1,5 @@
 PKG_NAME     := libfixmath
-PKG_VERSION  := 24488b16cc0359daada0682a9bd3a11a801d0a01
+PKG_VERSION  := b987044c73dcaba496bb5bc86e69d134cd8790ec
 PKG_URL      := https://github.com/PetteriAimonen/libfixmath
 PKG_LICENSE  := MIT
 
@@ -9,4 +9,4 @@ all: $(filter libfixmath-unittests,$(USEMODULE))
 	$(QQ)"$(MAKE)" -C $(PKG_SOURCE_DIR)/$(PKG_NAME) -f $(CURDIR)/Makefile.$(PKG_NAME)
 
 libfixmath-unittests:
-	$(QQ)"$(MAKE)" -C $(PKG_SOURCE_DIR)/unittests -f $(CURDIR)/Makefile.$(PKG_NAME)-unittests
+	$(QQ)"$(MAKE)" -C $(PKG_SOURCE_DIR)/tests -f $(CURDIR)/Makefile.$(PKG_NAME)-unittests

pkg/libfixmath/Makefile.include

@@ -14,5 +14,6 @@ CFLAGS += $(libfixmath_options:CONFIG_FIXMATH_%=-DFIXMATH_%)
 INCLUDES += -I$(PKG_SOURCE_DIR)/libfixmath
 
 ifneq (,$(filter libfixmath-unittests,$(USEMODULE)))
-  INCLUDES += -I$(PKG_SOURCE_DIR)/unittests
+  INCLUDES += -I$(PKG_SOURCE_DIR)
+  INCLUDES += -I$(PKG_SOURCE_DIR)/tests
 endif

pkg/tinydtls/Makefile

@@ -1,6 +1,6 @@
 PKG_NAME=tinydtls
 PKG_URL=https://github.com/eclipse/tinydtls.git
-PKG_VERSION=297fced854b652591b78113f0ba8d57ad9f934d9
+PKG_VERSION=c58f484ac417afe036273d65506b63a0902c740c
 PKG_LICENSE=EPL-1.0,EDL-1.0
 
 include $(RIOTBASE)/pkg/pkg.mk

pkg/tinydtls/Makefile.include

@@ -39,12 +39,6 @@ ifeq (,$(CONFIG_KCONFIG_USEPKG_TINYDTLS))
   endif
 endif
 
-ifneq (,$(or $(CONFIG_DTLS_DEBUG),$(filter -DCONFIG_DTLS_DEBUG,$(CFLAGS))))
-  CFLAGS += -DTINYDTLS_LOG_LVL=6
-else
-  CFLAGS += -DTINYDTLS_LOG_LVL=0
-endif
-
 # For now contrib only contains sock_dtls adaption
 ifneq (,$(filter tinydtls_sock_dtls,$(USEMODULE)))
   DIRS += $(RIOTBASE)/pkg/tinydtls/contrib

pkg/tinydtls/contrib/sock_dtls.c

@@ -887,7 +887,6 @@ void sock_dtls_close(sock_dtls_t *sock)
 void sock_dtls_init(void)
 {
     dtls_init();
-    dtls_set_log_level(TINYDTLS_LOG_LVL);
 }
 
 static void _ep_to_session(const sock_udp_ep_t *ep, session_t *session)

tests/pkg_libfixmath_unittests/Makefile

@@ -7,6 +7,11 @@ ifneq (,$(filter native,$(BOARD)))
 endif
 
 USEMODULE += printf_float
+USEMODULE += test_utils_main_exit_cb
+
+CFLAGS += -Wno-error=strict-prototypes
+CFLAGS += -Wno-error=old-style-definition
+CFLAGS += -Wno-error=format
 
 TEST_ON_CI_WHITELIST += native
 

tests/pkg_libfixmath_unittests/main.c

@@ -29,17 +29,12 @@
  */
 
 #include <stdio.h>
-#include "libfixmath-unittests.h"
 
-#define RUN(x)  if (x() != 0) { return 1; }
-
-int main(void)
+void test_utils_main_exit_cb(int res)
 {
-    RUN(fix16_exp_unittests);
-    RUN(fix16_macros_unittests);
-    RUN(fix16_str_unittests);
-    RUN(fix16_unittests);
-
-    puts("SUCCESS");
-    return 0;
+    if (res) {
+        puts("FAILED");
+    } else {
+        puts("SUCCESS");
+    }
 }