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 2021/12/23 12:14:21 UTC

[GitHub] [incubator-nuttx] acassis commented on pull request #5065: sched: use exec_spawn instead of exec, serial: add CONFIG_TTY_LAUNCH support

acassis commented on pull request #5065:
URL: https://github.com/apache/incubator-nuttx/pull/5065#issuecomment-1000264118


   @GUIDINGLI could you please provide more details of these modification? I noticed that CONFIG_INIT_ENTRYPOINT was renamed to CONFIG_INIT_ENTRY but the old name still referenced in many places. I suggest we test it more before incorporating it on mainline.
   
   Also the TTY_LAUNCH purpose is not too clear because currently we already have mechanisms to start NuttX using other TTY inputs (see CONFIG_NSH_ALTCONDEV). To avoid what Tim just called "Obscure Menuconfig" I suggest you to create a documentation about this feature to let people know how to use 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