Hi

I realize there is no context in that patch proposal, let me clarify a bit.
First, there is a typo in the first sed instruction, and therefore the #if
0 is never changed to #if 1 at the beginning of lv_conf.h.

Secondly I added an install append to copy the generated lv_conf.h as part
of the lvgl package, so that it will be found and used by the recipes that
want to use lvgl.

With these changes I have a working build for my lvgl app; I added some
additional changes in a bbappend of my own, but I didn't want to put too
many of these in the main lv-conf.inc file so I didn't propose it here.

Christophe

On Thu, Mar 14, 2024 at 7:28 PM Tofe via lists.openembedded.org
<chris.chapuis=gmail....@lists.openembedded.org> wrote:

> Signed-off-by: Christophe Chapuis <chris.chap...@gmail.com>
> ---
>  meta-oe/recipes-graphics/lvgl/lv-conf.inc | 7 ++++++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc
> b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
> index cb676ac62..cd14412e1 100644
> --- a/meta-oe/recipes-graphics/lvgl/lv-conf.inc
> +++ b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
> @@ -24,7 +24,7 @@ do_configure:append() {
>      # If there is a configuration template, start from that
>      [ -r "${S}/lv_conf_template.h" ] && cp -Lv "${S}/lv_conf_template.h"
> "${S}/lv_conf.h"
>
> -    sed -e "s|#if 0 .*Set it to \"1\" to enable the content.*|#if 1 //
> Enabled by ${PN}|g" \
> +    sed -e "s|#if 0 .*Set it to \"1\" to enable content.*|#if 1 //
> Enabled by ${PN}|g" \
>          \
>          -e "s|\(^#define LV_USE_LINUX_DRM \).*|#define LV_USE_LINUX_DRM
> ${LVGL_CONFIG_USE_DRM}|g" \
>          \
> @@ -52,3 +52,8 @@ do_configure:append() {
>          \
>          -i "${S}/lv_conf.h"
>  }
> +
> +do_install:append() {
> +    install -d "${D}${includedir}/lvgl"
> +    install -m 0644 "${S}/lv_conf.h" "${D}${includedir}/lvgl/lv_conf.h"
> +}
> --
> 2.44.0
>
>
> 
>
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109317): 
https://lists.openembedded.org/g/openembedded-devel/message/109317
Mute This Topic: https://lists.openembedded.org/mt/104932391/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to