yamt commented on PR #16022:
URL: https://github.com/apache/nuttx/pull/16022#issuecomment-2795305281

   > > > @yamt seems we have an issue on `toywasm`. Could you please take a 
look at the CI build log?
   > > 
   > > 
   > > is it still failing?
   > 
   > Yes. @fdcavalcanti is on leave and there are other defconfigs to be fixed. 
He fixed most of them, but the one related to toywasm is external, so that's 
why we pinged you.
   > 
   > It fails to build with the following errors:
   
   thank you. i will take a look later.
   
   > 
   > ```
   > % Total    % Received % Xferd  Average Speed   Time    Time     Time  
Current
   >                                  Dload  Upload   Total   Spent    Left  
Speed
   >   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--   
  0  % Total    % Received % Xferd  Average Speed   Time    Time     Time  
Current
   >                                  Dload  Upload   Total   Spent    Left  
Speed
   >   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--   
  0  % 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
   >   0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--   
  0
   > 100  496k  100  496k    0     0   
[85](https://gitlab.espressif.cn:6688/idf/apache/incubator-nuttx/-/jobs/63772390#L85)5k
      0 --:--:-- --:--:-- --:--:-- 1637k
   > 100  340k  100  340k    0     0   499k      0 --:--:-- --:--:-- --:--:--  
499k
   > 100  617k    0  617k    0     0   753k      0 --:--:-- --:--:-- --:--:--  
753k
   > warning: remote HEAD refers to nonexistent ref, unable to checkout.
   > toywasm/lib/exec_debug.c: In function 'exec_context_print_stats':
   > toywasm/lib/exec_debug.c:13:21: error: format '%zu' expects argument of 
type 'size_t', but argument 4 has type 'long unsigned int' [-Werror=format=]
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                     ^~~~~~
   >    14 |                     (vec)->psize * sizeof(*(vec)->p));
   >       |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   >       |                                  |
   >       |                                  long unsigned int
   > toywasm/lib/exec_debug.c:23:9: note: in expansion of macro 
'VEC_PRINT_USAGE'
   >    23 |         VEC_PRINT_USAGE("operand stack", &ctx->stack);
   >       |         ^~~~~~~~~~~~~~~
   > toywasm/lib/exec_debug.c:13:40: note: format string is defined here
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                                      ~~^
   >       |                                        |
   >       |                                        unsigned int
   >       |                                      %lu
   > toywasm/lib/exec_debug.c:13:21: error: format '%zu' expects argument of 
type 'size_t', but argument 4 has type 'long unsigned int' [-Werror=format=]
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                     ^~~~~~
   >    14 |                     (vec)->psize * sizeof(*(vec)->p));
   >       |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   >       |                                  |
   >       |                                  long unsigned int
   > toywasm/lib/exec_debug.c:25:9: note: in expansion of macro 
'VEC_PRINT_USAGE'
   >    25 |         VEC_PRINT_USAGE("locals", &ctx->locals);
   >       |         ^~~~~~~~~~~~~~~
   > toywasm/lib/exec_debug.c:13:40: note: format string is defined here
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                                      ~~^
   >       |                                        |
   >       |                                        unsigned int
   >       |                                      %lu
   > toywasm/lib/exec_debug.c:13:21: error: format '%zu' expects argument of 
type 'size_t', but argument 4 has type 'long unsigned int' [-Werror=format=]
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                     ^~~~~~
   >    14 |                     (vec)->psize * sizeof(*(vec)->p));
   >       |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   >       |                                  |
   >       |                                  long unsigned int
   > toywasm/lib/exec_debug.c:27:9: note: in expansion of macro 
'VEC_PRINT_USAGE'
   >    27 |         VEC_PRINT_USAGE("labels", &ctx->labels);
   >       |         ^~~~~~~~~~~~~~~
   > toywasm/lib/exec_debug.c:13:40: note: format string is defined here
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                                      ~~^
   >       |                                        |
   >       |                                        unsigned int
   >       |                                      %lu
   > toywasm/lib/exec_debug.c:13:21: error: format '%zu' expects argument of 
type 'size_t', but argument 4 has type 'long unsigned int' [-Werror=format=]
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                     ^~~~~~
   >    14 |                     (vec)->psize * sizeof(*(vec)->p));
   >       |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   >       |                                  |
   >       |                                  long unsigned int
   > toywasm/lib/exec_debug.c:28:9: note: in expansion of macro 
'VEC_PRINT_USAGE'
   >    28 |         VEC_PRINT_USAGE("frames", &ctx->frames);
   >       |         ^~~~~~~~~~~~~~~
   > toywasm/lib/exec_debug.c:13:40: note: format string is defined here
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                                      ~~^
   >       |                                        |
   >       |                                        unsigned int
   >       |                                      %lu
   > toywasm/lib/exec_debug.c:13:21: error: format '%zu' expects argument of 
type 'size_t', but argument 4 has type 'long unsigned int' [-Werror=format=]
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                     ^~~~~~
   >    14 |                     (vec)->psize * sizeof(*(vec)->p));
   >       |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   >       |                                  |
   >       |                                  long unsigned int
   > toywasm/lib/exec_debug.c:29:9: note: in expansion of macro 
'VEC_PRINT_USAGE'
   >    29 |         VEC_PRINT_USAGE("restarts", &ctx->restarts);
   >       |         ^~~~~~~~~~~~~~~
   > toywasm/lib/exec_debug.c:13:40: note: format string is defined here
   >    13 |         nbio_printf("%s %" PRIu32 " (%zu bytes)\n", (name), 
(vec)->psize,     \
   >       |                                      ~~^
   >       |                                        |
   >       |                                        unsigned int
   >       |                                      %lu
   > toywasm/lib/exec_insn_subr.c: In function 'memory_getptr2':
   > 
toywasm/lib/exec_insn_subr.c:[130](https://gitlab.espressif.cn:6688/idf/apache/incubator-nuttx/-/jobs/63772390#L130):25:
 error: format '%lu' expects argument of type 'long unsigned int', but argument 
9 has type 'int' [-Werror=format=]
   >   130 |                         "invalid memory access at %04" PRIx32 " 
%08" PRIx32
   >       |                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   > ......
   >   
[134](https://gitlab.espressif.cn:6688/idf/apache/incubator-nuttx/-/jobs/63772390#L134)
 |                         1 << memtype_page_shift(meminst->type));
   >       |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   >       |                           |
   >       |                           int
   > In file included from 
/builds/idf/apache/incubator-nuttx/include/stdint.h:37,
   >                  from 
/builds/idf/apache/incubator-nuttx/include/sys/types.h:34,
   >                  from 
/builds/idf/apache/incubator-nuttx/include/assert.h:33,
   >                  from toywasm/lib/exec_insn_subr.c:1:
   > /builds/idf/apache/incubator-nuttx/include/arch/inttypes.h:57:23: note: 
format string is defined here
   >    57 | #define PRIu32      "lu"
   > toywasm/lib/host_instance.c: In function 'host_func_getptr2':
   > toywasm/lib/host_instance.c:218:25: error: format '%lu' expects argument 
of type 'long unsigned int', but argument 7 has type 'int' [-Werror=format=]
   >   218 |                         "host function invalid memory access at 
%08" PRIx32
   >       |                         
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   > ......
   >   222 |                         1 << memtype_page_shift(mem->type));
   >       |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   >       |                           |
   >       |                           int
   > In file included from 
/builds/idf/apache/incubator-nuttx/include/stdint.h:37,
   >                  from 
/builds/idf/apache/incubator-nuttx/include/sys/types.h:34,
   >                  from 
/builds/idf/apache/incubator-nuttx/include/assert.h:33,
   >                  from toywasm/lib/host_instance.c:1:
   > /builds/idf/apache/incubator-nuttx/include/arch/inttypes.h:57:23: note: 
format string is defined here
   >    57 | #define PRIu32      "lu"
   > At top level:
   > cc1: note: unrecognized command-line option '-Wno-unknown-warning-option' 
may have been intended to silence earlier diagnostics
   > cc1: all warnings being treated as errors
   > At top level:
   > cc1: note: unrecognized command-line option '-Wno-unknown-warning-option' 
may have been intended to silence earlier diagnostics
   > cc1: all warnings being treated as errors
   > toywasm/lib/instance.c: In function 'instance_print_stats':
   > make[2]: *** [/builds/idf/apache/apps/Application.mk:239: 
exec_debug.c.builds.idf.apache.apps.interpreters.toywasm.o] Error 1
   > make[2]: *** Waiting for unfinished jobs....
   > make[2]: *** [/builds/idf/apache/apps/Application.mk:239: 
host_instance.c.builds.idf.apache.apps.interpreters.toywasm.o] Error 1
   > toywasm/lib/instance.c:780:29: error: format '%lu' expects argument of 
type 'long unsigned int', but argument 7 has type 'int' [-Werror=format=]
   >   780 |                 nbio_printf("memory[%" PRIu32
   >       |                             ^~~~~~~~~~
   > ......
   >   785 |                             lim->max, 1 << 
memtype_page_shift(mi->type));
   >       |                                       
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   >       |                                         |
   >       |                                         int
   > In file included from 
/builds/idf/apache/incubator-nuttx/include/stdint.h:37,
   >                  from 
/builds/idf/apache/incubator-nuttx/include/sys/types.h:34,
   >                  from 
/builds/idf/apache/incubator-nuttx/include/assert.h:33,
   >                  from toywasm/lib/instance.c:1:
   > /builds/idf/apache/incubator-nuttx/include/arch/inttypes.h:57:23: note: 
format string is defined here
   >    57 | #define PRIu32      "lu"
   > toywasm/lib/module.c: In function 'module_load_into':
   > toywasm/lib/module.c:2217:61: error: format '%u' expects argument of type 
'unsigned int', but argument 3 has type 'uint32_t' {aka 'long unsigned int'} 
[-Werror=format=]
   >  2217 |                 report_error(&ctx->report, "wrong version: %u", v);
   >       |                                                            ~^   ~
   >       |                                                             |   |
   >       |                                                             |   
uint32_t {aka long unsigned int}
   >       |                                                             
unsigned int
   >       |                                                            %lu
   > At top level:
   > cc1: note: unrecognized command-line option '-Wno-unknown-warning-option' 
may have been intended to silence earlier diagnostics
   > cc1: all warnings being treated as errors
   > make[2]: *** [/builds/idf/apache/apps/Application.mk:239: 
exec_insn_subr.c.builds.idf.apache.apps.interpreters.toywasm.o] Error 1
   > At top level:
   > cc1: note: unrecognized command-line option '-Wno-unknown-warning-option' 
may have been intended to silence earlier diagnostics
   > cc1: all warnings being treated as errors
   > make[2]: *** [/builds/idf/apache/apps/Application.mk:239: 
instance.c.builds.idf.apache.apps.interpreters.toywasm.o] Error 1
   > At top level:
   > cc1: note: unrecognized command-line option '-Wno-unknown-warning-option' 
may have been intended to silence earlier diagnostics
   > cc1: all warnings being treated as errors
   > make[2]: *** 
[/builds/idf/apache/apps/Application.mk:[239](https://gitlab.espressif.cn:6688/idf/apache/incubator-nuttx/-/jobs/63772390#L239):
 module.c.builds.idf.apache.apps.interpreters.toywasm.o] Error 1
   > make[1]: *** [Makefile:54: 
/builds/idf/apache/apps/interpreters/toywasm_all] Error 2
   > make: *** [tools/LibTargets.mk:248: /builds/idf/apache/apps/libapps.a] 
Error 2
   > ```
   
   


-- 
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: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to