Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Script to automatically pull LVGL updates #34

Open
Richard-Stump opened this issue Apr 10, 2023 · 0 comments · May be fixed by #45
Open

Script to automatically pull LVGL updates #34

Richard-Stump opened this issue Apr 10, 2023 · 0 comments · May be fixed by #45

Comments

@Richard-Stump
Copy link
Contributor

Overview

Although updating liblvgl is less cumbersome now that it is removed from the kernel, we still need to do the following to update liblvgl:

  • Copy all the header files from LVLG's repo to include/liblvgl
  • Copy all the source files from LVLG's repo to src/liblvgl
  • Update all the includes in the LVGL files to match the PROS project structure
  • Update include/liblvgl/fonts/lv_font.h to include our custom fonts that llemu depends on

My Proposal

Fixing the font issue:

  1. Store the fonts that llemu depends on in a vector graphics file
  2. On build, the makefile creates the lvgl font.c files
  3. The fonts are declared in a new pros_line.h file so we don't have to add then to lv_font.h

Automated update script:

The script should not run automatically when LVGL updates, since we want to force someone to make sure everything works and builds properly

These are the things I think the script needs to do:

  1. Check the latest release tag on LVGL's repo. If we are up to date, do nothing
  2. Otherwise, clone the repo as a submodule
  3. Copy all the headers to include/libvgl
  4. Copy all the c files to src/liblvgl
  5. Run through all the copied files and update the includes to be absolute and follow the PROS project structure.
@unwieldycat unwieldycat linked a pull request Jun 22, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant