You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by "masayuki2009 (via GitHub)" <gi...@apache.org> on 2023/08/13 11:30:25 UTC

[GitHub] [nuttx-apps] masayuki2009 opened a new pull request, #1962: apps: Replace CONFIG_ARCH_HAVE_VFORK with CONFIG_ARCH_HAVE_FORK

masayuki2009 opened a new pull request, #1962:
URL: https://github.com/apache/nuttx-apps/pull/1962

   ## Summary
   
   - I noticed that vfork_test is not excecuted since https://github.com/apache/nuttx/pull/9755 renamed CONFIG_ARCH_HAVE_VFORK to CONFIG_ARCH_HAVE_FORK
   - This PR fixes this issue
   
   ## Impact
   
   - NOTE: the above PR causes the vfork_test crash with aarch64 now,
     so please ** do not merge ** this PR until the crash is fixed.
   
   ## Testing
   
   - Tested with rv-virt:smp64, spresense:smp, sim:smp
   


-- 
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-unsubscribe@nuttx.apache.org

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


Re: [PR] apps: Replace CONFIG_ARCH_HAVE_VFORK with CONFIG_ARCH_HAVE_FORK [nuttx-apps]

Posted by "xiaoxiang781216 (via GitHub)" <gi...@apache.org>.
xiaoxiang781216 merged PR #1962:
URL: https://github.com/apache/nuttx-apps/pull/1962


-- 
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-unsubscribe@nuttx.apache.org

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


Re: [PR] apps: Replace CONFIG_ARCH_HAVE_VFORK with CONFIG_ARCH_HAVE_FORK [nuttx-apps]

Posted by "masayuki2009 (via GitHub)" <gi...@apache.org>.
masayuki2009 commented on PR #1962:
URL: https://github.com/apache/nuttx-apps/pull/1962#issuecomment-1813766301

   Please ignore the following style issues.
   
   ```
   Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/interpreters/bas/bas_statement.c:53:7: error: Mixed case identifier found
   Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/interpreters/bas/bas_statement.c:53:14: error: Mixed case identifier found
   Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/interpreters/bas/bas_statement.c:53:31: error: Mixed case identifier found
   Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/interpreters/bas/bas_statement.c:58:13: error: Mixed case identifier found
   Error: /home/runner/work/nuttx-apps/nuttx-apps/apps/interpreters/bas/bas_statement.c:69:10: error: Mixed case identifier found
   E
   ...
   ```


-- 
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-unsubscribe@nuttx.apache.org

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