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

fix(CMakeLists): add options to select backend #52

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

js214
Copy link

@js214 js214 commented Jul 26, 2024

Add options USE_FBDEV, USE_DRM, USE_SDL2 to CMakeLists.txt to select the backend. Also change lv_conf.h to make fbdev the optional default.

Add option USE_THOR to CMakeLists.txt so as to make using thorvg optional. This way C++ is not required for building. Also change lv_conf.h to make building the internal ThorVG optional.

Mark lvgl as EXCLUDE_FROM_ALL in CMakeLists.txt so it does not get installed on the target when lv_port_linux is installed.

These changes are useful to enable including this package in buildroot (see issue #17).

js214 added 2 commits July 26, 2024 12:23
Add options `USE_FBDEV`, `USE_DRM`, `USE_SDL2` to CMakeLists.txt to
select the backend. Also change `lv_conf.h` to make fbdev the optional
default.

Add option `USE_THOR` to CMakeLists.txt so as to make using thorvg
optional.  This way C++ is not required for building. Also change
`lv_conf.h` to make building the internal ThorVG optional.

Mark lvgl as `EXCLUDE_FROM_ALL` in CMakeLists.txt so it does not get
installed on the target when `lv_port_linux` is installed.

These changes are useful to enable including this package in buildroot
(see issue lvgl#17).

Signed-off-by: Jakob Kastelic <[email protected]>
In bd4545c ('add options to select backend') the default option is to
use the framebuffer. When `USE_SDL2=ON` or `USE_DRM=ON` are passed to
CMake, the framebuffer option must be turned off. This is fixed with the
present commit.

Also, in order for these options to apply to lvgl,
`add_subdirectory(lvgl ...)` needs to be moved after the option setting
code.

Signed-off-by: Jakob Kastelic <[email protected]>
@YobeZhou
Copy link
Contributor

Hi,

Please check here: https://github.com/lvgl/lv_port_linux/pull/51/files

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 this pull request may close these issues.

2 participants