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

   ## Summary
   The lvglterm terminal now runs the shell on a pseudo-terminal instead of 
three plain pipes.
   Reason for this change is that I'm trying to run Python on LVGL terminal and 
we need to use pseudo terminal for it to render the interactive mode.
   
   - apps/examples/lvglterm/lvglterm.c — create_terminal() now calls openpty() 
and gives the shell the slave end as stdin/stdout/stderr; the single master fd 
replaces the three pipe pairs
   - New `lvglterm_send_input()` in the core owns the write path. It only ever 
writes as much as the terminal reports free via FIONSPACE and queues the rest 
for the timer.
   - Both input variants no longer echo locally (lvglterm_touch.c, 
lvglterm_kbd.c) — the terminal driver echoes the characters as the shell reads 
them, exactly as on a UART console, and echoing again would double everything.
   - Kconfig gained select PSEUDOTERM, so no defconfig edits are needed.
   
   ### AI Disclaimer
   Used Cursor to execute the changes and evaluate the impact.
   
   ## Impact
   
   Should affect the following boards that have `lvgl_term` example:
   - esp32p4-tab5
   - esp32s3-m5-cardputer
   - linum-stm32h753bi
   
   ## Testing
   Works the same as before.
   Tested on `esp32p4-tab5` and it works great. Need feedback for the other 
board.
   
   @JorgeGzm could you do a test run with keyboard and touch? Which I think are 
the ones affected on linum board.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to