Skip to content

system/nxstore: Add LVGL touchscreen app-store frontend. - #3643

Merged
acassis merged 1 commit into
apache:masterfrom
aviralgarg05:gsoc/nxstore-app-store-ui-pr6
Aug 18, 2026
Merged

system/nxstore: Add LVGL touchscreen app-store frontend.#3643
acassis merged 1 commit into
apache:masterfrom
aviralgarg05:gsoc/nxstore-app-store-ui-pr6

Conversation

@aviralgarg05

@aviralgarg05 aviralgarg05 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

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, and git diff --check
  • built NXStore and nxpkg against current master
  • installed and launched a package on hardware
  • closed it through the supervisor bar and confirmed the updated list returned

PR verification Self-Check

  • This PR introduces one focused change.
  • I have updated all required description fields above.
  • I have reviewed and signed every commit.
  • This PR adheres to the current contribution and coding guidelines.
  • My PR is still work in progress.
  • My PR is ready for review and can be safely merged.

@acassis

acassis commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

@aviralgarg05 nice work, please include a scheenshot in the Testing. Also please add a companion nuttx/Documentation

@aviralgarg05
aviralgarg05 force-pushed the gsoc/nxstore-app-store-ui-pr6 branch from 513edde to 79daf2c Compare July 24, 2026 10:01
@aviralgarg05 aviralgarg05 changed the title system/nxstore: add LVGL touchscreen app-store frontend system/nxstore: Add LVGL touchscreen app-store frontend. Jul 24, 2026
@aviralgarg05

Copy link
Copy Markdown
Contributor Author

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.

@acassis

acassis commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

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

@aviralgarg05

Copy link
Copy Markdown
Contributor Author

@aviralgarg05 please double check, there is not screenshot in apache/nuttx#18875

Sorry, missed it, Done now

@acassis

acassis commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

@aviralgarg05 please double check, there is not screenshot in apache/nuttx#18875

Sorry, missed it, Done now

don't worry

@aviralgarg05
aviralgarg05 force-pushed the gsoc/nxstore-app-store-ui-pr6 branch from 79daf2c to e93ea02 Compare August 14, 2026 15:12
@aviralgarg05
aviralgarg05 marked this pull request as ready for review August 14, 2026 15:14
@aviralgarg05
aviralgarg05 force-pushed the gsoc/nxstore-app-store-ui-pr6 branch 2 times, most recently from e37eeeb to 7badb69 Compare August 14, 2026 15:51
@github-actions

Copy link
Copy Markdown

🔗 Cross-repo PR dependencies

The 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 linguini1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread system/nxstore/nxstore_main.c Outdated
Comment thread system/nxstore/nxstore_main.c Outdated
Comment thread system/nxstore/nxstore_main.c Outdated
@linguini1

Copy link
Copy Markdown
Contributor

On the board: the catalog was listed, a package was installed and launched by tapping it, closing it sent SIGTERM and the application exited on its
own, and the list came back with the package now shown as installed.

Show this. Text description is not enough.

@aviralgarg05
aviralgarg05 force-pushed the gsoc/nxstore-app-store-ui-pr6 branch from e5a969a to 68f50e7 Compare August 15, 2026 10:02
@aviralgarg05

Copy link
Copy Markdown
Contributor Author

Show this. Text description is not enough.

Will record a video of it, and put up asap

@github-actions

Copy link
Copy Markdown

🔗 Cross-repo PR dependencies

The 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

@linguini1

Copy link
Copy Markdown
Contributor

Even just photos is fine

Comment thread system/nxstore/Kconfig Outdated
@xiaoxiang781216

Copy link
Copy Markdown
Contributor

@aviralgarg05 squash you change into one patch.

@aviralgarg05
aviralgarg05 force-pushed the gsoc/nxstore-app-store-ui-pr6 branch from 76204e7 to a4fe5a1 Compare August 16, 2026 12:15
@aviralgarg05

Copy link
Copy Markdown
Contributor Author

@linguini1

IMG_0787-github.1.mp4

The video shows catalog -> install and launch -> cooperative Close -> updated installed catalog
The evidence is a plain black screen just for reference to show the working of flow

@aviralgarg05
aviralgarg05 force-pushed the gsoc/nxstore-app-store-ui-pr6 branch from a4fe5a1 to 2c6b64d Compare August 16, 2026 15:31
@github-actions

Copy link
Copy Markdown

🔗 Cross-repo PR dependencies

The 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

Comment thread include/system/nxstore_chrome.h Outdated
Comment thread system/nxstore/Kconfig Outdated
Comment thread system/nxstore/nxstore_main.c Outdated
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>
@aviralgarg05
aviralgarg05 force-pushed the gsoc/nxstore-app-store-ui-pr6 branch from 2c6b64d to fb9b994 Compare August 17, 2026 13:23

@cederom cederom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @aviralgarg05 very nice idea, documentation is welcome :-)

@acassis
acassis merged commit 0eb44a1 into apache:master Aug 18, 2026
63 of 92 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants