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 2022/04/22 09:14:31 UTC

[GitHub] [incubator-nuttx] pussuw opened a new pull request, #6131: RISC-V: Add implementation for vfork

pussuw opened a new pull request, #6131:
URL: https://github.com/apache/incubator-nuttx/pull/6131

   ## Summary
   Add vfork for risc-v
   ## Impact
   None, unless vfork is used
   ## Testing
   user_main: vfork() test
   vfork_test: Child 153 ran successfully
   
   


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


[GitHub] [incubator-nuttx] pussuw commented on pull request #6131: RISC-V: Add implementation for vfork

Posted by GitBox <gi...@apache.org>.
pussuw commented on PR #6131:
URL: https://github.com/apache/incubator-nuttx/pull/6131#issuecomment-1106420968

   I think this CI error is fixed now:
    Error: unistd/lib_execle.c:175:9: error: too many arguments to function 'execv'
     175 |   ret = execv(path, argv, envp);
         |         ^~~~~
         
   Can someone e.g. @pkarashchenko re-start the CI test ? Or if I can do it tell me how to do it ?


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


[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6131: RISC-V: Add implementation for vfork

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged PR #6131:
URL: https://github.com/apache/incubator-nuttx/pull/6131


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


[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6131: RISC-V: Add implementation for vfork

Posted by GitBox <gi...@apache.org>.
pkarashchenko commented on PR #6131:
URL: https://github.com/apache/incubator-nuttx/pull/6131#issuecomment-1106426970

   > I think this CI error is fixed now: Error: unistd/lib_execle.c:175:9: error: too many arguments to function 'execv' 175 | ret = execv(path, argv, envp); | ^~~~~
   > 
   > Can someone e.g. @pkarashchenko re-start the CI test ? Or if I can do it tell me how to do it ?
   
   please rebase on top of the latest master. This should be fixed by https://github.com/apache/incubator-nuttx/pull/6130


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


[GitHub] [incubator-nuttx] pussuw commented on pull request #6131: RISC-V: Add implementation for vfork

Posted by GitBox <gi...@apache.org>.
pussuw commented on PR #6131:
URL: https://github.com/apache/incubator-nuttx/pull/6131#issuecomment-1106461839

   Tried rebase via gh ui... I'll wipe the merge commit in a few hours...


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