search

jakpaul / lvgl_touch_calibration

A complete lvgl screen object for performing calibration of a resistive touch panel.
MIT License
24 stars 8 forks source link

LV_LVGL_H_INCLUDE_SIMPLE should be changed #4

Closed ZoranIT closed 1 year ago

ZoranIT commented 1 year ago

I'm trying to compile this calibration with lvgl and esp_lcd_touch in eclipse. lvgl/lvgl.h does not exist in 8.3.9 shouldn't it be removed and changed to only lvgl.h? I can't find a way to add LV_LVGL_H_INCLUDE_SIMPLE flag in eclipse.

https://github.com/jakpaul/lvgl_touch_calibration/blob/7326c0b29d5bc5eb9ce1695a81096b6a735d551e/img/lv_tc_indicator_img.c#L1C1-L6C1

Thanks,

jakpaul commented 1 year ago

This appears to be a lvgl feature which lets you select from where the library should be included. See here: CMake - LVGL Docs.

The file was originally generated using the image converter tool on lvgl.io. Upon regenerating it now, some new macros have been added at the beginning of the file. If your compiler supports __has_include, it should fix the issue by defaulting to "lvgl.h" instead of "lvgl/lvgl.h" for the include path. I made a new branch with the changes. Feel free to test this out and let me know if it works.

ZoranIT commented 1 year ago

Okay, thank you. This variant compiles with eclipse, esp-idf and dependenciesover the yml file.

jakpaul commented 1 year ago

You're welcome. Will be merging this into master