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

Converted font won't compile as-is #41

Closed
rbarris opened this issue Dec 14, 2019 · 2 comments
Closed

Converted font won't compile as-is #41

rbarris opened this issue Dec 14, 2019 · 2 comments

Comments

@rbarris
Copy link

rbarris commented Dec 14, 2019

We use GitHub issues for development related discussions.
Please use the forum to ask questions.

Describe the issue
I used the LittlevGL online font converter tool to convert a font.
I downloaded the resulting C file and added it to my project incorporating the lv_port_esp32_ili9341.
The font source isn't fully compatible with this version of LittlevGL and had errors I had to fix.
It is not actually clear to me whether this is an issue in the font converter being too new, or the code in this port being different, not sure. The errors are minor, here is the diff

rbarris@oryx-i7:~/sandbox/ray/esp1$ diff ~/Downloads/build.c ~/sandbox/ray/esp1/main/build.c
1377c1377
< .class_pair_values = kern_class_values,

.class_pair_values   = (const uint8_t *)kern_class_values,

1412c1412
< .subpx = LV_FONT_SUBPX_NONE,

// .subpx = LV_FONT_SUBPX_NONE,

Code to reproduce the issue
font edits.zip

Expected Results
A converted font should integrate smoothly with LVGL

Actual Results
minor compile errors

ESP32 Chip version
m5stack gray

ESP-IDF version
ESP-IDF v4.1-dev-1086-g93a8603c5-dirty

Development machine OS
Ubuntu 17.10

@embeddedt
Copy link
Member

This is because the font converter has been updated to support LittlevGL 6.1, but there are a few minor incompatibilities with 6.0.x now. There are two possible solutions:

  • Fix the minor incompatibility the way you did, and continue to use LittlevGL 6.0.
  • Upgrade to LittlevGL 6.1; it's a minor release so you shouldn't need to do much work to upgrade.

@kisvegabor
Copy link
Member

As the question is answered I close this issue.

NickXia007 pushed a commit to NickXia007/lv_port_esp32 that referenced this issue Jul 23, 2023
Remove AXP192 symbols being created when it shouldn't
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

No branches or pull requests

3 participants