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/03/22 04:05:49 UTC

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on issue #131: Remove support for CONFIG_FS_WRITABLE

xiaoxiang781216 commented on issue #131: Remove support for CONFIG_FS_WRITABLE
URL: https://github.com/apache/incubator-nuttx-apps/pull/131#issuecomment-602145014
 
 
   > @xiaoxiang781216 Yes, I verified the problem. These apps/ changes will not build without the corresponding nuttx/ changes in place. Right now, the PR Check CI / build (arm-14) is failing on the nucleo-f091rc:nsh.
   > 
   
   I saw some nxstyle errors, do you want to fix them before merge? or let me merge this PR directly.
   
   > However, I just built that configuration with no errors or warnings of any kind when both the apps/ changes and the nuttx/ changes are combined. We need a solution for this.
   
   @liuguo09, could you find some solution to fix this issue?
   Before we get a fixed, we have to ignore this false alarm.

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


With regards,
Apache Git Services