fdcavalcanti commented on PR #3200:
URL: https://github.com/apache/nuttx-apps/pull/3200#issuecomment-3494874174
> @fdcavalcanti errors with
>
> * same70-xplained:mcuboot-loader
>
> * same70-qmtech/mcuboot-loader
>
>
> ```
>
====================================================================================
> Configuration/Tool:
same70-xplained/mcuboot-loader,CONFIG_ARM_TOOLCHAIN_GNU_EABI
> 2025-11-05 12:19:08
>
------------------------------------------------------------------------------------
> Cleaning...
> Configuring...
> Disabling CONFIG_ARM_TOOLCHAIN_GNU_EABI
> Enabling CONFIG_ARM_TOOLCHAIN_GNU_EABI
> Building NuttX...
> % Total % Received % Xferd Average Speed Time Time Time
Current
> Dload Upload Total Spent Left
Speed
>
> 0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:--
0
> 0 0 0 0 0 0 0 0 --:--:-- --:--:-- --:--:--
0
>
> 100 1807k 0 1807k 0 0 3662k 0 --:--:-- --:--:-- --:--:--
3662k
> In file included from
/github/workspace/sources/apps/boot/mcuboot/mcuboot/boot/bootutil/include/bootutil/crypto/sha.h:25,
> from mcuboot/boot/bootutil/src/bootutil_img_hash.c:32:
> mcuboot/boot/bootutil/src/bootutil_img_hash.c: In function
'bootutil_img_hash':
> Error:
/github/workspace/sources/apps/boot/mcuboot/mcuboot/boot/nuttx/include/mcuboot_config/mcuboot_logging.h:46:23:
error: format '%u' expects argument of type 'unsigned int', but argument 6 has
type 'uint32_t' {aka 'long unsigned int'} [-Werror=format=]
> 46 | syslog(LOG_DEBUG, "%s: " format "\n", __FUNCTION__,
##__VA_ARGS__)
> | ^~~~~~
>
/github/workspace/sources/apps/boot/mcuboot/mcuboot/boot/bootutil/include/bootutil/bootutil_log.h:36:27:
note: in expansion of macro 'MCUBOOT_LOG_DBG'
> 36 | #define BOOT_LOG_DBG(...) MCUBOOT_LOG_DBG(__VA_ARGS__)
> | ^~~~~~~~~~~~~~~
> mcuboot/boot/bootutil/src/bootutil_img_hash.c:173:13: note: in expansion
of macro 'BOOT_LOG_DBG'
> 173 | BOOT_LOG_DBG("bootutil_img_validate Error %d reading
data chunk %p %u %u",
> | ^~~~~~~~~~~~
> Error:
/github/workspace/sources/apps/boot/mcuboot/mcuboot/boot/nuttx/include/mcuboot_config/mcuboot_logging.h:46:23:
error: format '%u' expects argument of type 'unsigned int', but argument 7 has
type 'uint32_t' {aka 'long unsigned int'} [-Werror=format=]
> 46 | syslog(LOG_DEBUG, "%s: " format "\n", __FUNCTION__,
##__VA_ARGS__)
> | ^~~~~~
>
/github/workspace/sources/apps/boot/mcuboot/mcuboot/boot/bootutil/include/bootutil/bootutil_log.h:36:27:
note: in expansion of macro 'MCUBOOT_LOG_DBG'
> 36 | #define BOOT_LOG_DBG(...) MCUBOOT_LOG_DBG(__VA_ARGS__)
> | ^~~~~~~~~~~~~~~
> mcuboot/boot/bootutil/src/bootutil_img_hash.c:173:13: note: in expansion
of macro 'BOOT_LOG_DBG'
> 173 | BOOT_LOG_DBG("bootutil_img_validate Error %d reading
data chunk %p %u %u",
> | ^~~~~~~~~~~~
> cc1: all warnings being treated as errors
> make[2]: *** [/github/workspace/sources/apps/Application.mk:239:
mcuboot/boot/bootutil/src/bootutil_img_hash.c.github.workspace.sources.apps.boot.mcuboot.o]
Error 1
> In file included from
/github/workspace/sources/apps/boot/mcuboot/mcuboot/boot/bootutil/include/bootutil/bootutil_log.h:31,
> from mcuboot/boot/bootutil/src/bootutil_misc.c:40:
> mcuboot/boot/bootutil/src/bootutil_misc.c: In function 'boot_erase_region':
> Error:
/github/workspace/sources/apps/boot/mcuboot/mcuboot/boot/nuttx/include/mcuboot_config/mcuboot_logging.h:46:23:
error: format '%d' expects argument of type 'int', but argument 5 has type
'uint32_t' {aka 'long unsigned int'} [-Werror=format=]
> 46 | syslog(LOG_DEBUG, "%s: " format "\n", __FUNCTION__,
##__VA_ARGS__)
> | ^~~~~~
>
/github/workspace/sources/apps/boot/mcuboot/mcuboot/boot/bootutil/include/bootutil/bootutil_log.h:36:27:
note: in expansion of macro 'MCUBOOT_LOG_DBG'
> 36 | #define BOOT_LOG_DBG(...) MCUBOOT_LOG_DBG(__VA_ARGS__)
> | ^~~~~~~~~~~~~~~
> ```
>
> [Linux
(arm-07)](https://github.com/apache/nuttx-apps/actions/runs/19054806631/job/54563922291#logs)
Seems those warnings are being treated as errors, just a type mismatching.
This can be fixed (hidden) by disabling CONFIG_MCUBOOT_ENABLE_LOGGING on the
defconfig, only those two defconfigs are using it.
The alternative is a patch on MCUBoot, which I'm unable to do. You guys okay
with the defconfig change?
Otherwise a fix should
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
