You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2020/11/29 21:31:18 UTC

[GitHub] [incubator-nuttx-apps] btashton edited a comment on pull request #510: LVGL: use NuttX's printf()

btashton edited a comment on pull request #510:
URL: https://github.com/apache/incubator-nuttx-apps/pull/510#issuecomment-735456605


   @v01d Looks like it is more than just the sign issue with the format string (uint32 vs int).  I did this change and everything built fine (on top of your lvgl improvement).
   ```c
   diff --git a/graphics/lvgl/Make.defs b/graphics/lvgl/Make.defs
   index 681b8f4f..96c203a5 100644
   --- a/graphics/lvgl/Make.defs
   +++ b/graphics/lvgl/Make.defs
   @@ -36,6 +36,7 @@
    ifeq ($(CONFIG_GRAPHICS_LVGL),y)
    CONFIGURED_APPS += $(APPDIR)/graphics/lvgl
    
   +CFLAGS += "-Wno-format"
    # It allows `<lvgl/lvgl.h>` import.
   ```
   ```
   ./tools/configure.sh stm32f429i-disco:lvgl
   make EXTRAFLAGS="-Wno-cpp -Werror"
   ```
   
   That is the underlying make argument that the CI system uses btw.


----------------------------------------------------------------
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.

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