system/nxstore: Add LVGL touchscreen app-store frontend. - #3643
Conversation
ce6c2e5 to
513edde
Compare
|
@aviralgarg05 nice work, please include a scheenshot in the Testing. Also please add a companion nuttx/Documentation |
513edde to
79daf2c
Compare
|
Thanks. I have added a photograph of NXStore running on the ESP32-S3 target to the Testing section. The companion documentation is in apache/nuttx#18875, and the later signed-tree build and serial evidence remain recorded in the PR description. |
@aviralgarg05 please double check, there is not screenshot in apache/nuttx#18875 |
Sorry, missed it, Done now |
don't worry |
79daf2c to
e93ea02
Compare
e37eeeb to
7badb69
Compare
🔗 Cross-repo PR dependenciesThe read-only Build run reported the following dependent PR(s) and fetched head SHA(s): CI run: https://github.com/apache/nuttx-apps/actions/runs/31816523839 |
linguini1
left a comment
There was a problem hiding this comment.
Honestly not going to finish reviewing this unless more effort is put into these patches to review AI output. These comments are way too long, some of the ones submitted before have been unrelated, and this is a consistent issue.
Show this. Text description is not enough. |
e5a969a to
68f50e7
Compare
Will record a video of it, and put up asap |
🔗 Cross-repo PR dependenciesThe read-only Build run reported the following dependent PR(s) and fetched head SHA(s): CI run: https://github.com/apache/nuttx-apps/actions/runs/31879473762 |
|
Even just photos is fine |
|
@aviralgarg05 squash you change into one patch. |
76204e7 to
a4fe5a1
Compare
IMG_0787-github.1.mp4The video shows catalog -> install and launch -> cooperative Close -> updated installed catalog |
a4fe5a1 to
2c6b64d
Compare
🔗 Cross-repo PR dependenciesThe read-only Build run reported the following dependent PR(s) and fetched head SHA(s): CI run: https://github.com/apache/nuttx-apps/actions/runs/31955926952 |
Add an LVGL front end for browsing, installing, updating, and launching nxpkg entries. Keep network and install work off the UI path, and stop launched apps with SIGTERM. Signed-off-by: aviralgarg05 <gargaviral99@gmail.com>
2c6b64d to
fb9b994
Compare
cederom
left a comment
There was a problem hiding this comment.
Thank you @aviralgarg05 very nice idea, documentation is welcome :-)
Summary
Add a small LVGL front end for browsing and managing nxpkg entries. NXStore
uses the public
<system/nxpkg.h>API and keeps slow work off the UI thread.NXStore supervises each launched application. Closing an app sends SIGTERM;
the package list returns after the child exits.
Impact
Testing
Host: macOS 26.5, arm64,
xtensa-esp-elf-gcc 14.2.0.Target: Waveshare ESP32-S3-Touch-LCD-7 with an 800x480 RGB565 display and
GT911 touchscreen.
nxstyle,tools/checkpatch.sh,codespell, andgit diff --checkmasterPR verification Self-Check