Re: [oe] [meta-oe][PATCH v3 5/5] lvgl: add more variables to lv-conf.inc

2024-03-15 Thread Marek Vasut

On 3/15/24 7:03 PM, Christophe Chapuis wrote:

As it can be usefull to customize these other variables, let's
add them in lv-conf.inc.

Signed-off-by: Christophe Chapuis 
---
  meta-oe/recipes-graphics/lvgl/lv-conf.inc | 17 +
  1 file changed, 17 insertions(+)

diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc 
b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
index f5a93dcae..088842490 100644
--- a/meta-oe/recipes-graphics/lvgl/lv-conf.inc
+++ b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
@@ -12,8 +12,17 @@ LVGL_CONFIG_USE_SDL = "${@bb.utils.contains('PACKAGECONFIG', 
'sdl', '1', '0', d)
  LVGL_CONFIG_SDL_FULLSCREEN ?= "0"
  
  LVGL_CONFIG_LV_MEM_CUSTOM ?= "0"

+LVGL_CONFIG_LV_MEM_SIZE ?= "(64 * 1024U)"
  LVGL_CONFIG_LV_COLOR_DEPTH ?= "32"
  
+LVGL_CONFIG_LV_USE_LOG?= "0"

+LVGL_CONFIG_LV_LOG_LEVEL  ?= "LV_LOG_LEVEL_WARN"
+LVGL_CONFIG_LV_LOG_PRINTF ?= "0"
+
+LVGL_CONFIG_LV_USE_FONT_COMPRESSED ?= "0"
+
+LVGL_CONFIG_LV_THEME_DEFAULT_DARK ?= "0"
+
  DEBUG_BUILD ??= "0"
  
  ALLOW_EMPTY:${PN} = "1"

@@ -37,6 +46,7 @@ do_configure:append() {
  \
  -e "s|\(^#define LV_COLOR_DEPTH 
\).*|\1${LVGL_CONFIG_LV_COLOR_DEPTH}|g" \
  -e "s|\(#define LV_MEM_CUSTOM 
.*\).*|\1${LVGL_CONFIG_LV_MEM_CUSTOM}|g" \
+-e "s|\(^ \+#define LV_MEM_SIZE \).*|\1${LVGL_CONFIG_LV_MEM_SIZE}|g" \
  \
  -e "s|\(#define LV_TICK_CUSTOM \).*|\1 1|g" \
  -e "s|\(#define LV_TICK_CUSTOM_INCLUDE \).*|\1 |g" \
@@ -50,6 +60,13 @@ do_configure:append() {
  -e "s|\(^#define LV_USE_ASSERT_MEM_INTEGRITY \).*|\1${DEBUG_BUILD}|g" 
\
  -e "s|\(^#define LV_USE_ASSERT_OBJ \).*|\1${DEBUG_BUILD}|g" \
  \
+-e "s|\(^#define LV_USE_LOG \).*|\1${LVGL_CONFIG_LV_USE_LOG}|g" \
+-e "s|\(^ \+#define LV_LOG_LEVEL \).*|\1${LVGL_CONFIG_LV_LOG_LEVEL}|g" 
\
+-e "s|\(^ \+#define LV_LOG_PRINTF 
\).*|\1${LVGL_CONFIG_LV_LOG_PRINTF}|g" \
+\
+-e "s|\(^#define LV_USE_FONT_COMPRESSED 
\).*|\1${LVGL_CONFIG_LV_USE_FONT_COMPRESSED}|g" \
+-e "s|\(^ \+#define LV_THEME_DEFAULT_DARK 
\).*|\1${LVGL_CONFIG_LV_THEME_DEFAULT_DARK}|g" \


Similar concerns as in 4/5 , but aside from that:

Reviewed-by: Marek Vasut 

Thanks !

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109351): 
https://lists.openembedded.org/g/openembedded-devel/message/109351
Mute This Topic: https://lists.openembedded.org/mt/104953145/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [oe] [meta-oe][PATCH v3 4/5] lvgl: cleanup sed instructions in lv-conf.inc

2024-03-15 Thread Marek Vasut

On 3/15/24 7:03 PM, Christophe Chapuis wrote:

Use back reference instead of repeating the search pattern. Later on
this could lead to a more generic way of replacing values in the .h file.

Signed-off-by: Christophe Chapuis 
---
  meta-oe/recipes-graphics/lvgl/lv-conf.inc | 28 +++
  1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc 
b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
index 6bcead628..f5a93dcae 100644
--- a/meta-oe/recipes-graphics/lvgl/lv-conf.inc
+++ b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
@@ -26,29 +26,29 @@ do_configure:append() {
  
  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" \
+-e "s|\(^#define LV_USE_LINUX_DRM \).*|\1${LVGL_CONFIG_USE_DRM}|g" \
  \
--e "s|\(^#define LV_USE_LINUX_FBDEV \).*|#define LV_USE_LINUX_FBDEV 
${LVGL_CONFIG_USE_FBDEV}|g" \
+-e "s|\(^#define LV_USE_LINUX_FBDEV \).*|\1${LVGL_CONFIG_USE_FBDEV}|g" 
\
  \
--e "s|\(^#define LV_USE_SDL \).*|#define LV_USE_SDL 
${LVGL_CONFIG_USE_SDL}|g" \
--e "s|\(^#define LV_USE_DRAW_SDL \).*|#define LV_USE_DRAW_SDL 
${LVGL_CONFIG_USE_SDL}|g" \
--e "s|\(^#define LV_SDL_BUF_COUNT \).*|#define LV_SDL_BUF_COUNT 
2|g" \
--e "s|\(^#define LV_SDL_FULLSCREEN \).*|#define LV_SDL_FULLSCREEN 
${LVGL_CONFIG_SDL_FULLSCREEN}|g" \
+-e "s|\(^#define LV_USE_SDL \).*|\1${LVGL_CONFIG_USE_SDL}|g" \
+-e "s|\(^#define LV_USE_DRAW_SDL \).*|\1${LVGL_CONFIG_USE_SDL}|g" \
+-e "s|\(^ \+#define LV_SDL_BUF_COUNT \).*|\1 2|g" \
+-e "s|\(^ \+#define LV_SDL_FULLSCREEN 
\).*|\1${LVGL_CONFIG_SDL_FULLSCREEN}|g" \


Wouldn't it be better to use '^ *#define...' here to catch the case 
where there are no spaces between start of line and #define ?



--e "s|\(^#define LV_COLOR_DEPTH \).*|#define LV_COLOR_DEPTH 
${LVGL_CONFIG_LV_COLOR_DEPTH}|g" \
--e "s|\(#define LV_MEM_CUSTOM .*\)0|\1${LVGL_CONFIG_LV_MEM_CUSTOM}|g" \
+-e "s|\(^#define LV_COLOR_DEPTH 
\).*|\1${LVGL_CONFIG_LV_COLOR_DEPTH}|g" \
+-e "s|\(#define LV_MEM_CUSTOM .*\).*|\1${LVGL_CONFIG_LV_MEM_CUSTOM}|g" 
\
  \
  -e "s|\(#define LV_TICK_CUSTOM \).*|\1 1|g" \
  -e "s|\(#define LV_TICK_CUSTOM_INCLUDE \).*|\1 |g" \
  -e "s|\(#define LV_TICK_CUSTOM_SYS_TIME_EXPR \).*|extern uint32_t 
custom_tick_get(void);\n\1 (custom_tick_get())|g" \
  \
--e "s|\(^#define LV_USE_EVDEV \).*|#define LV_USE_EVDEV 
${LVGL_CONFIG_USE_EVDEV}|g" \
+-e "s|\(^#define LV_USE_EVDEV \).*|\1${LVGL_CONFIG_USE_EVDEV}|g" \
  \
--e "s|\(^#define LV_USE_ASSERT_NULL \).*|#define LV_USE_ASSERT_NULL 
${DEBUG_BUILD}|g" \
--e "s|\(^#define LV_USE_ASSERT_MALLOC \).*|#define LV_USE_ASSERT_MALLOC 
${DEBUG_BUILD}|g" \
--e "s|\(^#define LV_USE_ASSERT_STYLE \).*|#define LV_USE_ASSERT_STYLE 
${DEBUG_BUILD}|g" \
--e "s|\(^#define LV_USE_ASSERT_MEM_INTEGRITY \).*|#define 
LV_USE_ASSERT_MEM_INTEGRITY ${DEBUG_BUILD}|g" \
--e "s|\(^#define LV_USE_ASSERT_OBJ \).*|#define LV_USE_ASSERT_OBJ 
${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_NULL \).*|\1${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_MALLOC \).*|\1${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_STYLE \).*|\1${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_MEM_INTEGRITY \).*|\1${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_OBJ \).*|\1${DEBUG_BUILD}|g" \


btw if you want this to be extra accurate, you could do also:

"s|\(^#define \\).*|\1 ${DEBUG_BUILD}|g"

which would handle '#define LV_USE_ASSERT_OBJ' as well .

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109350): 
https://lists.openembedded.org/g/openembedded-devel/message/109350
Mute This Topic: https://lists.openembedded.org/mt/104953143/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [oe] [meta-oe][PATCH v3 2/5] lvgl: install lv_conf.h

2024-03-15 Thread Marek Vasut

On 3/15/24 7:03 PM, Christophe Chapuis wrote:

Add 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.

Signed-off-by: Christophe Chapuis 


Reviewed-by: Marek Vasut 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109348): 
https://lists.openembedded.org/g/openembedded-devel/message/109348
Mute This Topic: https://lists.openembedded.org/mt/104953141/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [oe] [meta-oe][PATCH v3 3/5] lvgl: remove useless FILES include

2024-03-15 Thread Marek Vasut

On 3/15/24 7:03 PM, Christophe Chapuis wrote:

As ${includedir}${PN} is already listed in FILES, there is no
need to add one of its subdirectories.

Signed-off-by: Christophe Chapuis 


Reviewed-by: Marek Vasut 

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109349): 
https://lists.openembedded.org/g/openembedded-devel/message/109349
Mute This Topic: https://lists.openembedded.org/mt/104953142/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [oe] [meta-oe][PATCH v3 1/5] lvgl: fix typo in lv-conf.inc

2024-03-15 Thread Marek Vasut

On 3/15/24 7:03 PM, Christophe Chapuis wrote:

Fix a typo in the first sed instruction, where the #if 0 is
never changed to #if 1 at the beginning of lv_conf.h.

Signed-off-by: Christophe Chapuis 
---
  meta-oe/recipes-graphics/lvgl/lv-conf.inc | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc 
b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
index cb676ac62..3dbdf769f 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" \


Just to be on the safe side, can you please check if this also modifies 
the lv_conf.h in lvgl-demo-fb recipe correctly ? (I think it does not, 
but the fix is something like:


sed -e "s|#if 0 .*Set it to \"1\" to enable.*content.*|#if 1 // Enabled 
by ${PN}|g" \

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109347): 
https://lists.openembedded.org/g/openembedded-devel/message/109347
Mute This Topic: https://lists.openembedded.org/mt/104953140/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [meta-networking][PATCH 2/2] frr: Upgrade to latest on 9.1 stable

2024-03-15 Thread Khem Raj
Fix build with latest musl

Signed-off-by: Khem Raj 
---
 ...me-API-for-non-glibc-library-e.g.-mu.patch | 35 +++
 .../recipes-protocols/frr/frr_9.1.bb  |  3 +-
 2 files changed, 37 insertions(+), 1 deletion(-)
 create mode 100644 
meta-networking/recipes-protocols/frr/frr/0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch

diff --git 
a/meta-networking/recipes-protocols/frr/frr/0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch
 
b/meta-networking/recipes-protocols/frr/frr/0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch
new file mode 100644
index 00..215d6b4137
--- /dev/null
+++ 
b/meta-networking/recipes-protocols/frr/frr/0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch
@@ -0,0 +1,35 @@
+From 73c21bafc7065f0ab1d40fee2c6ce0c140d1b859 Mon Sep 17 00:00:00 2001
+From: Khem Raj 
+Date: Fri, 15 Mar 2024 11:31:02 -0700
+Subject: [PATCH] Mimic GNU basename() API for non-glibc library e.g. musl
+
+musl only provides POSIX version of basename and it has also removed
+providing it via string.h header [1] which now results in compile errors
+with newer compilers e.g. clang-18
+
+[1] 
https://git.musl-libc.org/cgit/musl/commit/?id=725e17ed6dff4d0cd22487bb64470881e86a92e7
+
+Upstream-Status: Submitted [https://github.com/FRRouting/frr/pull/15561]
+Signed-off-by: Khem Raj 
+---
+ lib/zebra.h | 4 
+ 1 file changed, 4 insertions(+)
+
+diff --git a/lib/zebra.h b/lib/zebra.h
+index 15a54f6cdf..cc40b48339 100644
+--- a/lib/zebra.h
 b/lib/zebra.h
+@@ -226,6 +226,10 @@ struct in_pktinfo {
+ 
+ #define strmatch(a,b) (!strcmp((a), (b)))
+ 
++#if !defined(__GLIBC__)
++#define basename(src) (strrchr(src,'/') ? strrchr(src,'/')+1 : src)
++#endif
++
+ #if BYTE_ORDER == LITTLE_ENDIAN
+ #define htonll(x) (((uint64_t)htonl((x)&0x) << 32) | htonl((x) >> 32))
+ #define ntohll(x) (((uint64_t)ntohl((x)&0x) << 32) | ntohl((x) >> 32))
+-- 
+2.44.0
+
diff --git a/meta-networking/recipes-protocols/frr/frr_9.1.bb 
b/meta-networking/recipes-protocols/frr/frr_9.1.bb
index 8ea917afaa..4510c27060 100644
--- a/meta-networking/recipes-protocols/frr/frr_9.1.bb
+++ b/meta-networking/recipes-protocols/frr/frr_9.1.bb
@@ -12,9 +12,10 @@ LIC_FILES_CHKSUM = 
"file://doc/licenses/GPL-2.0;md5=b234ee4d69f5fce4486a80fdaf4a
 
 SRC_URI = "git://github.com/FRRouting/frr.git;protocol=https;branch=stable/9.1 
\
file://frr.pam \
+   
file://0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch \
"
 
-SRCREV = "312faf8008bb4f3b9e84b8e2758cd2cbdf5742b5"
+SRCREV = "ca2d6f0f1e000951224a18973cc1827f7f5215b5"
 
 UPSTREAM_CHECK_GITTAGREGEX = "frr-(?P\d+(\.\d+)+)$"
 
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109346): 
https://lists.openembedded.org/g/openembedded-devel/message/109346
Mute This Topic: https://lists.openembedded.org/mt/104954208/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [meta-oe][PATCH 1/2] dietsplash: Update and fix build with musl

2024-03-15 Thread Khem Raj
Signed-off-by: Khem Raj 
---
 ...me-API-for-non-glibc-library-e.g.-mu.patch | 28 +++
 .../dietsplash/dietsplash_git.bb  |  7 +++--
 2 files changed, 32 insertions(+), 3 deletions(-)
 create mode 100644 
meta-oe/recipes-graphics/dietsplash/dietsplash/0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch

diff --git 
a/meta-oe/recipes-graphics/dietsplash/dietsplash/0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch
 
b/meta-oe/recipes-graphics/dietsplash/dietsplash/0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch
new file mode 100644
index 00..aa8406e315
--- /dev/null
+++ 
b/meta-oe/recipes-graphics/dietsplash/dietsplash/0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch
@@ -0,0 +1,28 @@
+From 272491297564513c4ce49a11bc1ecc523a3afd63 Mon Sep 17 00:00:00 2001
+From: Khem Raj 
+Date: Fri, 15 Mar 2024 11:05:54 -0700
+Subject: [PATCH] Mimic GNU basename() API for non-glibc library e.g. musl
+
+Upstream-Status: Submitted [https://github.com/lucasdemarchi/dietsplash/pull/7]
+Signed-off-by: Khem Raj 
+---
+ src/util.h | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/src/util.h b/src/util.h
+index 1f4c3ef..a3ab54a 100644
+--- a/src/util.h
 b/src/util.h
+@@ -60,6 +60,9 @@
+  */
+ #define ARRAY_SIZE(arr) (sizeof(arr) / sizeof((arr)[0]) + 
_array_size_chk(arr))
+ 
++#if !defined(__GLIBC__)
++#define basename(src) (strrchr(src,'/') ? strrchr(src,'/')+1 : src)
++#endif
+ 
+ #define DIE_PREFIX "[" PACKAGE_NAME "] ERR: "
+ #define LOG_SUFFIX "\n"
+-- 
+2.44.0
+
diff --git a/meta-oe/recipes-graphics/dietsplash/dietsplash_git.bb 
b/meta-oe/recipes-graphics/dietsplash/dietsplash_git.bb
index bd94b9ad51..1ee2657bd0 100644
--- a/meta-oe/recipes-graphics/dietsplash/dietsplash_git.bb
+++ b/meta-oe/recipes-graphics/dietsplash/dietsplash_git.bb
@@ -4,12 +4,13 @@ LIC_FILES_CHKSUM = 
"file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 
 # Really, no depends besides a C library
 
-PV = "0.3"
+PV = "0.3+git"
 
-SRCREV = "ef2e1a390e768e21e6a6268977580ee129a96633"
+SRCREV = "8bed71d860bbb0c8792fa2a1179c9beeae84d577"
 SRC_URI = 
"git://github.com/lucasdemarchi/dietsplash.git;branch=master;protocol=https \
file://0001-configure.ac-Do-not-demand-linker-hash-style.patch \
-   "
+   
file://0001-Mimic-GNU-basename-API-for-non-glibc-library-e.g.-mu.patch \
+"
 
 inherit autotools
 
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109345): 
https://lists.openembedded.org/g/openembedded-devel/message/109345
Mute This Topic: https://lists.openembedded.org/mt/104954207/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [meta-oe][PATCH v3 5/5] lvgl: add more variables to lv-conf.inc

2024-03-15 Thread Tofe
As it can be usefull to customize these other variables, let's
add them in lv-conf.inc.

Signed-off-by: Christophe Chapuis 
---
 meta-oe/recipes-graphics/lvgl/lv-conf.inc | 17 +
 1 file changed, 17 insertions(+)

diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc 
b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
index f5a93dcae..088842490 100644
--- a/meta-oe/recipes-graphics/lvgl/lv-conf.inc
+++ b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
@@ -12,8 +12,17 @@ LVGL_CONFIG_USE_SDL = "${@bb.utils.contains('PACKAGECONFIG', 
'sdl', '1', '0', d)
 LVGL_CONFIG_SDL_FULLSCREEN ?= "0"
 
 LVGL_CONFIG_LV_MEM_CUSTOM ?= "0"
+LVGL_CONFIG_LV_MEM_SIZE ?= "(64 * 1024U)"
 LVGL_CONFIG_LV_COLOR_DEPTH ?= "32"
 
+LVGL_CONFIG_LV_USE_LOG?= "0"
+LVGL_CONFIG_LV_LOG_LEVEL  ?= "LV_LOG_LEVEL_WARN"
+LVGL_CONFIG_LV_LOG_PRINTF ?= "0"
+
+LVGL_CONFIG_LV_USE_FONT_COMPRESSED ?= "0"
+
+LVGL_CONFIG_LV_THEME_DEFAULT_DARK ?= "0"
+
 DEBUG_BUILD ??= "0"
 
 ALLOW_EMPTY:${PN} = "1"
@@ -37,6 +46,7 @@ do_configure:append() {
 \
 -e "s|\(^#define LV_COLOR_DEPTH 
\).*|\1${LVGL_CONFIG_LV_COLOR_DEPTH}|g" \
 -e "s|\(#define LV_MEM_CUSTOM .*\).*|\1${LVGL_CONFIG_LV_MEM_CUSTOM}|g" 
\
+-e "s|\(^ \+#define LV_MEM_SIZE \).*|\1${LVGL_CONFIG_LV_MEM_SIZE}|g" \
 \
 -e "s|\(#define LV_TICK_CUSTOM \).*|\1 1|g" \
 -e "s|\(#define LV_TICK_CUSTOM_INCLUDE \).*|\1 |g" \
@@ -50,6 +60,13 @@ do_configure:append() {
 -e "s|\(^#define LV_USE_ASSERT_MEM_INTEGRITY \).*|\1${DEBUG_BUILD}|g" \
 -e "s|\(^#define LV_USE_ASSERT_OBJ \).*|\1${DEBUG_BUILD}|g" \
 \
+-e "s|\(^#define LV_USE_LOG \).*|\1${LVGL_CONFIG_LV_USE_LOG}|g" \
+-e "s|\(^ \+#define LV_LOG_LEVEL \).*|\1${LVGL_CONFIG_LV_LOG_LEVEL}|g" 
\
+-e "s|\(^ \+#define LV_LOG_PRINTF 
\).*|\1${LVGL_CONFIG_LV_LOG_PRINTF}|g" \
+\
+-e "s|\(^#define LV_USE_FONT_COMPRESSED 
\).*|\1${LVGL_CONFIG_LV_USE_FONT_COMPRESSED}|g" \
+-e "s|\(^ \+#define LV_THEME_DEFAULT_DARK 
\).*|\1${LVGL_CONFIG_LV_THEME_DEFAULT_DARK}|g" \
+\
 -i "${S}/lv_conf.h"
 }
 
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109344): 
https://lists.openembedded.org/g/openembedded-devel/message/109344
Mute This Topic: https://lists.openembedded.org/mt/104953145/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [meta-oe][PATCH v3 4/5] lvgl: cleanup sed instructions in lv-conf.inc

2024-03-15 Thread Tofe
Use back reference instead of repeating the search pattern. Later on
this could lead to a more generic way of replacing values in the .h file.

Signed-off-by: Christophe Chapuis 
---
 meta-oe/recipes-graphics/lvgl/lv-conf.inc | 28 +++
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc 
b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
index 6bcead628..f5a93dcae 100644
--- a/meta-oe/recipes-graphics/lvgl/lv-conf.inc
+++ b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
@@ -26,29 +26,29 @@ do_configure:append() {
 
 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" \
+-e "s|\(^#define LV_USE_LINUX_DRM \).*|\1${LVGL_CONFIG_USE_DRM}|g" \
 \
--e "s|\(^#define LV_USE_LINUX_FBDEV \).*|#define LV_USE_LINUX_FBDEV 
${LVGL_CONFIG_USE_FBDEV}|g" \
+-e "s|\(^#define LV_USE_LINUX_FBDEV \).*|\1${LVGL_CONFIG_USE_FBDEV}|g" 
\
 \
--e "s|\(^#define LV_USE_SDL \).*|#define LV_USE_SDL 
${LVGL_CONFIG_USE_SDL}|g" \
--e "s|\(^#define LV_USE_DRAW_SDL \).*|#define LV_USE_DRAW_SDL 
${LVGL_CONFIG_USE_SDL}|g" \
--e "s|\(^#define LV_SDL_BUF_COUNT \).*|#define 
LV_SDL_BUF_COUNT 2|g" \
--e "s|\(^#define LV_SDL_FULLSCREEN \).*|#define 
LV_SDL_FULLSCREEN ${LVGL_CONFIG_SDL_FULLSCREEN}|g" \
+-e "s|\(^#define LV_USE_SDL \).*|\1${LVGL_CONFIG_USE_SDL}|g" \
+-e "s|\(^#define LV_USE_DRAW_SDL \).*|\1${LVGL_CONFIG_USE_SDL}|g" \
+-e "s|\(^ \+#define LV_SDL_BUF_COUNT \).*|\1 2|g" \
+-e "s|\(^ \+#define LV_SDL_FULLSCREEN 
\).*|\1${LVGL_CONFIG_SDL_FULLSCREEN}|g" \
 \
--e "s|\(^#define LV_COLOR_DEPTH \).*|#define LV_COLOR_DEPTH 
${LVGL_CONFIG_LV_COLOR_DEPTH}|g" \
--e "s|\(#define LV_MEM_CUSTOM .*\)0|\1${LVGL_CONFIG_LV_MEM_CUSTOM}|g" \
+-e "s|\(^#define LV_COLOR_DEPTH 
\).*|\1${LVGL_CONFIG_LV_COLOR_DEPTH}|g" \
+-e "s|\(#define LV_MEM_CUSTOM .*\).*|\1${LVGL_CONFIG_LV_MEM_CUSTOM}|g" 
\
 \
 -e "s|\(#define LV_TICK_CUSTOM \).*|\1 1|g" \
 -e "s|\(#define LV_TICK_CUSTOM_INCLUDE \).*|\1 |g" \
 -e "s|\(#define LV_TICK_CUSTOM_SYS_TIME_EXPR \).*|extern uint32_t 
custom_tick_get(void);\n\1 (custom_tick_get())|g" \
 \
--e "s|\(^#define LV_USE_EVDEV \).*|#define LV_USE_EVDEV 
${LVGL_CONFIG_USE_EVDEV}|g" \
+-e "s|\(^#define LV_USE_EVDEV \).*|\1${LVGL_CONFIG_USE_EVDEV}|g" \
 \
--e "s|\(^#define LV_USE_ASSERT_NULL \).*|#define LV_USE_ASSERT_NULL 
${DEBUG_BUILD}|g" \
--e "s|\(^#define LV_USE_ASSERT_MALLOC \).*|#define 
LV_USE_ASSERT_MALLOC ${DEBUG_BUILD}|g" \
--e "s|\(^#define LV_USE_ASSERT_STYLE \).*|#define LV_USE_ASSERT_STYLE 
${DEBUG_BUILD}|g" \
--e "s|\(^#define LV_USE_ASSERT_MEM_INTEGRITY \).*|#define 
LV_USE_ASSERT_MEM_INTEGRITY ${DEBUG_BUILD}|g" \
--e "s|\(^#define LV_USE_ASSERT_OBJ \).*|#define LV_USE_ASSERT_OBJ 
${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_NULL \).*|\1${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_MALLOC \).*|\1${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_STYLE \).*|\1${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_MEM_INTEGRITY \).*|\1${DEBUG_BUILD}|g" \
+-e "s|\(^#define LV_USE_ASSERT_OBJ \).*|\1${DEBUG_BUILD}|g" \
 \
 -i "${S}/lv_conf.h"
 }
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109343): 
https://lists.openembedded.org/g/openembedded-devel/message/109343
Mute This Topic: https://lists.openembedded.org/mt/104953143/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [meta-oe][PATCH v3 2/5] lvgl: install lv_conf.h

2024-03-15 Thread Tofe
Add 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.

Signed-off-by: Christophe Chapuis 
---
 meta-oe/recipes-graphics/lvgl/lv-conf.inc | 5 +
 1 file changed, 5 insertions(+)

diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc 
b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
index 3dbdf769f..6bcead628 100644
--- a/meta-oe/recipes-graphics/lvgl/lv-conf.inc
+++ b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
@@ -52,3 +52,8 @@ do_configure:append() {
 \
 -i "${S}/lv_conf.h"
 }
+
+do_install:append() {
+install -d "${D}${includedir}/${PN}"
+install -m 0644 "${S}/lv_conf.h" "${D}${includedir}/${PN}/lv_conf.h"
+}
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109341): 
https://lists.openembedded.org/g/openembedded-devel/message/109341
Mute This Topic: https://lists.openembedded.org/mt/104953141/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [meta-oe][PATCH v3 3/5] lvgl: remove useless FILES include

2024-03-15 Thread Tofe
As ${includedir}${PN} is already listed in FILES, there is no
need to add one of its subdirectories.

Signed-off-by: Christophe Chapuis 
---
 meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb | 1 -
 1 file changed, 1 deletion(-)

diff --git a/meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb 
b/meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb
index b84fc5a3d..0b7eba538 100644
--- a/meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb
+++ b/meta-oe/recipes-graphics/lvgl/lvgl_9.0.0.bb
@@ -28,7 +28,6 @@ require lv-conf.inc
 
 FILES:${PN}-dev += "\
 ${includedir}/${PN}/ \
-${includedir}/${PN}/lvgl/ \
 "
 SOLIBS = ".so"
 FILES_SOLIBSDEV = ""
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109342): 
https://lists.openembedded.org/g/openembedded-devel/message/109342
Mute This Topic: https://lists.openembedded.org/mt/104953142/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [meta-oe][PATCH v3 1/5] lvgl: fix typo in lv-conf.inc

2024-03-15 Thread Tofe
Fix a typo in the first sed instruction, where the #if 0 is
never changed to #if 1 at the beginning of lv_conf.h.

Signed-off-by: Christophe Chapuis 
---
 meta-oe/recipes-graphics/lvgl/lv-conf.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc 
b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
index cb676ac62..3dbdf769f 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" \
 \
-- 
2.44.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109340): 
https://lists.openembedded.org/g/openembedded-devel/message/109340
Mute This Topic: https://lists.openembedded.org/mt/104953140/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [meta-openembedded] [PATCH 2/2] buildtools-imagemagick: Add new recipe

2024-03-15 Thread Richard Purdie
This is useful on the Yocto Project autobuilder to allow automated
screenshot testing/processing during QA.

Signed-off-by: Richard Purdie 
---
 meta-oe/recipes-core/meta/buildtools-imagemagick.bb | 10 ++
 1 file changed, 10 insertions(+)
 create mode 100644 meta-oe/recipes-core/meta/buildtools-imagemagick.bb

diff --git a/meta-oe/recipes-core/meta/buildtools-imagemagick.bb 
b/meta-oe/recipes-core/meta/buildtools-imagemagick.bb
new file mode 100644
index 00..d59f7a7a52
--- /dev/null
+++ b/meta-oe/recipes-core/meta/buildtools-imagemagick.bb
@@ -0,0 +1,10 @@
+require recipes-core/meta/buildtools-tarball.bb
+
+SUMMARY = "Standalone tarball of imagemagick binaries"
+LICENSE = "MIT"
+
+# Add nativesdk equivalent of build-essentials
+TOOLCHAIN_HOST_TASK = "nativesdk-imagemagick nativesdk-sdk-provides-dummy"
+TOOLCHAIN_OUTPUTNAME = 
"${SDK_ARCH}-buildtools-imagemagick-nativesdk-standalone-${DISTRO_VERSION}"
+
+SDK_TITLE = "Imagemagick tools"
-- 
2.40.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109339): 
https://lists.openembedded.org/g/openembedded-devel/message/109339
Mute This Topic: https://lists.openembedded.org/mt/104952248/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [meta-openembedded] [PATCH 1/2] imagemagick/lcms/fftw: Allow nativesdk versions to exist

2024-03-15 Thread Richard Purdie
These allow the use of imagemagick in SDKs where it is useful to perform
image analysis such as on the autobuilder for screenshot processing
during QA testing.

Signed-off-by: Richard Purdie 
---
 meta-oe/recipes-support/fftw/fftw_3.3.10.bb  | 2 +-
 meta-oe/recipes-support/imagemagick/imagemagick_7.1.1.bb | 2 +-
 meta-oe/recipes-support/lcms/lcms_2.16.bb| 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/meta-oe/recipes-support/fftw/fftw_3.3.10.bb 
b/meta-oe/recipes-support/fftw/fftw_3.3.10.bb
index 33e8279880..af7117da7b 100644
--- a/meta-oe/recipes-support/fftw/fftw_3.3.10.bb
+++ b/meta-oe/recipes-support/fftw/fftw_3.3.10.bb
@@ -100,4 +100,4 @@ RDEPENDS:${PN}-dev = "libfftw libfftwl libfftwf"
 RDEPENDS:${PN}-ptest += "perl"
 RDEPENDS:${PN}-ptest:remove = "fftw"
 
-BBCLASSEXTEND = "native"
+BBCLASSEXTEND = "native nativesdk"
diff --git a/meta-oe/recipes-support/imagemagick/imagemagick_7.1.1.bb 
b/meta-oe/recipes-support/imagemagick/imagemagick_7.1.1.bb
index dcc951bce8..6ab8a61b9b 100644
--- a/meta-oe/recipes-support/imagemagick/imagemagick_7.1.1.bb
+++ b/meta-oe/recipes-support/imagemagick/imagemagick_7.1.1.bb
@@ -52,7 +52,7 @@ FILES:${PN}-dev += 
"${libdir}/ImageMagick-${BASE_PV}/modules-Q16/*/*.a"
 
 FILES:${PN}-dbg += "${libdir}/ImageMagick-${BASE_PV}/modules-Q16/*/.debug/*"
 
-BBCLASSEXTEND = "native"
+BBCLASSEXTEND = "native nativesdk"
 
 ALTERNATIVE_PRIORITY = "100"
 
diff --git a/meta-oe/recipes-support/lcms/lcms_2.16.bb 
b/meta-oe/recipes-support/lcms/lcms_2.16.bb
index 10c1994a7e..8135f83a05 100644
--- a/meta-oe/recipes-support/lcms/lcms_2.16.bb
+++ b/meta-oe/recipes-support/lcms/lcms_2.16.bb
@@ -8,7 +8,7 @@ SRC_URI[sha256sum] = 
"d873d34ad8b9b4cea010631f1a6228d2087475e4dc5e763eb81acc23d9
 
 DEPENDS = "tiff"
 
-BBCLASSEXTEND = "native"
+BBCLASSEXTEND = "native nativesdk"
 
 S = "${WORKDIR}/lcms2-${PV}"
 
-- 
2.40.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109338): 
https://lists.openembedded.org/g/openembedded-devel/message/109338
Mute This Topic: https://lists.openembedded.org/mt/104952247/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



[oe] [PATCH][meta-python] python3-pydantic-core: just set PYPI_PACKAGE

2024-03-15 Thread Ross Burton
From: Ross Burton 

There's no need to set PYPI_ARCHIVE_NAME and S explicitly if PYPI_PACKAGE
is set correctly.

Signed-off-by: Ross Burton 
---
 .../recipes-devtools/python/python3-pydantic-core_2.16.3.bb   | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git 
a/meta-python/recipes-devtools/python/python3-pydantic-core_2.16.3.bb 
b/meta-python/recipes-devtools/python/python3-pydantic-core_2.16.3.bb
index b26478451a..2086091d63 100644
--- a/meta-python/recipes-devtools/python/python3-pydantic-core_2.16.3.bb
+++ b/meta-python/recipes-devtools/python/python3-pydantic-core_2.16.3.bb
@@ -17,9 +17,7 @@ require ${BPN}-crates.inc
 
 inherit pypi cargo-update-recipe-crates python_maturin
 
-S = "${WORKDIR}/pydantic_core-${PV}"
-
-PYPI_ARCHIVE_NAME = "pydantic_core-${PV}.${PYPI_PACKAGE_EXT}"
+PYPI_PACKAGE = "pydantic_core"
 
 RDEPENDS:${PN} += "python3-typing-extensions"
 
-- 
2.34.1


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109337): 
https://lists.openembedded.org/g/openembedded-devel/message/109337
Mute This Topic: https://lists.openembedded.org/mt/104945514/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [oe] [meta-oe][PATCHv2 3/3] lvgl: add more variables to lv-conf.inc

2024-03-15 Thread Marek Vasut

On 3/15/24 9:16 AM, Christophe Chapuis wrote:

Sure, I agree with your comments; I'll propose a patch v3 tonight or during
the week-end, taking all this into consideration.


Thank you !

I'll CC you on the LVGL 9.1 update once that is available too.

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109336): 
https://lists.openembedded.org/g/openembedded-devel/message/109336
Mute This Topic: https://lists.openembedded.org/mt/104933491/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [oe] [meta-oe][PATCHv2 3/3] lvgl: add more variables to lv-conf.inc

2024-03-15 Thread Tofe
Sure, I agree with your comments; I'll propose a patch v3 tonight or during
the week-end, taking all this into consideration.

On Fri, Mar 15, 2024 at 1:34 AM Marek Vasut  wrote:

> On 3/14/24 8:21 PM, Christophe Chapuis wrote:
> > As it can be usefull to customize these other variables, let's
> > add them in lv-conf.inc.
> >
> > Also, fix the install path to use ${PN} instead of "lvgl".
>
> Separate commit would be nice.
>
> btw when sending V3, please CC Khem and co. if you didn't yet.
>
> Also, I'll likely be sending LVGL 9.1 update next week, it should be out
> then.
>
> > Signed-off-by: Christophe Chapuis 
> > ---
> >   meta-oe/recipes-graphics/lvgl/lv-conf.inc | 49 +++
> >   1 file changed, 33 insertions(+), 16 deletions(-)
> >
> > diff --git a/meta-oe/recipes-graphics/lvgl/lv-conf.inc
> b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
> > index cd14412e1..088842490 100644
> > --- a/meta-oe/recipes-graphics/lvgl/lv-conf.inc
> > +++ b/meta-oe/recipes-graphics/lvgl/lv-conf.inc
> > @@ -12,8 +12,17 @@ LVGL_CONFIG_USE_SDL =
> "${@bb.utils.contains('PACKAGECONFIG', 'sdl', '1', '0', d)
> >   LVGL_CONFIG_SDL_FULLSCREEN ?= "0"
> >
> >   LVGL_CONFIG_LV_MEM_CUSTOM ?= "0"
> > +LVGL_CONFIG_LV_MEM_SIZE ?= "(64 * 1024U)"
> >   LVGL_CONFIG_LV_COLOR_DEPTH ?= "32"
> >
> > +LVGL_CONFIG_LV_USE_LOG?= "0"
> > +LVGL_CONFIG_LV_LOG_LEVEL  ?= "LV_LOG_LEVEL_WARN"
> > +LVGL_CONFIG_LV_LOG_PRINTF ?= "0"
> > +
> > +LVGL_CONFIG_LV_USE_FONT_COMPRESSED ?= "0"
> > +
> > +LVGL_CONFIG_LV_THEME_DEFAULT_DARK ?= "0"
> > +
> >   DEBUG_BUILD ??= "0"
> >
> >   ALLOW_EMPTY:${PN} = "1"
> > @@ -26,34 +35,42 @@ do_configure:append() {
> >
> >   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" \
> > +-e "s|\(^#define LV_USE_LINUX_DRM
> \).*|\1${LVGL_CONFIG_USE_DRM}|g" \
>
> I'd say, make this clean up a separate patch, otherwise this is
> difficult to review.
>
> Then, add the extra features on top.
>
> Thanks !
>

-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#109335): 
https://lists.openembedded.org/g/openembedded-devel/message/109335
Mute This Topic: https://lists.openembedded.org/mt/104933491/21656
Group Owner: openembedded-devel+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-



Re: [oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

2024-03-15 Thread Jörg Sommer via lists . openembedded . org
Yes, I'm getting these errors, too, when compiling with clang; independent of 
the clang version up to 19. It seems to be a problem of `pow()`. gcc might have 
a built-in and can compute it at compile-time, while clang has no built-in of 
pow.




Jörg Sommer

Software Developer / Programmierer

--


Navimatix GmbH

Tatzendpromenade 2

07745 Jena

T: 03641 - 327 99 0

F: 03641 - 526 306

M: joerg.som...@navimatix.de

www.navimatix.de



Geschäftsführer: Steffen Späthe, Jan Rommeley

Registergericht: Amtsgericht Jena, HRB 501480


From: Khem Raj 
Sent: Friday, 15 March 2024 06:13
To: Jörg Sommer 
Cc: openembedded-devel@lists.openembedded.org 

Subject: Re: [oe] [meta-multimedia][PATCH v2] spandsp: new telephony DSP library

Not much to do with musl, it fails when I use clang on my desktop as
well. you can see the same errors with clang on host

CC=clang ./configure
make

On Thu, Mar 14, 2024 at 9:37 PM Khem Raj  wrote:
>
> fails to build with musl/clang
>
> https://snips.sh/f/qCR86Rwwz1
>
> On Thu, Mar 14, 2024 at 12:43 PM Jörg Sommer via
> lists.openembedded.org
>  wrote:
> >
> > From: Jörg Sommer 
> >
> > Signed-off-by: Jörg Sommer 
> > ---
> >  .../spandsp/spandsp/configure.patch   |  56 ++
> >  .../spandsp/spandsp/makefile.patch| 161 ++
> >  .../recipes-multimedia/spandsp/spandsp_git.bb |  47 +
> >  3 files changed, 264 insertions(+)
> >  create mode 100644 
> > meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
> >  create mode 100644 
> > meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
> >  create mode 100644 
> > meta-multimedia/recipes-multimedia/spandsp/spandsp_git.bb
> >
> > diff --git 
> > a/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch 
> > b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
> > new file mode 100644
> > index 0..af504be76
> > --- /dev/null
> > +++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/configure.patch
> > @@ -0,0 +1,56 @@
> > +From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
> > +Date: Thu, 14 Mar 2024 09:41:57 +0100
> > +Subject: [PATCH 2/2] configure: Replace manual search for libxml by
> > + AC_CHECK_LIB
> > +MIME-Version: 1.0
> > +Content-Type: text/plain; charset=UTF-8
> > +Content-Transfer-Encoding: 8bit
> > +
> > +The manual search fails if the include directories (and build environment)
> > +are located somewhere else.
> > +
> > +Signed-off-by: Jörg Sommer 
> > +Upstream-Status: Submitted [https://github.com/freeswitch/spandsp/pull/74]
> > +---
> > +
> > +ERROR: spandsp-3.0.0+git-r0 do_configure: QA Issue: This autoconf log 
> > indicates errors, it looked at host include and/or library paths while 
> > determining system capabilities.
> > +
> > + configure.ac | 22 +-
> > + 1 file changed, 5 insertions(+), 17 deletions(-)
> > +
> > +diff --git a/configure.ac b/configure.ac
> > +index 2140837..b5138d6 100644
> > +--- a/configure.ac
> >  b/configure.ac
> > +@@ -185,23 +185,11 @@ then
> > + fi
> > +
> > + # Determine XML2 include path
> > +-AC_MSG_CHECKING(for libxml/xmlmemory.h)
> > +-
> > +-# Can we include headers using system include dirs?
> > +-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include ]], 
> > [[int a = 1;]])],[XML2_INCLUDE=" "],[XML2_INCLUDE=])
> > +-
> > +-# Hunt through several possible directories to find the includes for 
> > libxml2
> > +-if test "x$XML2_INCLUDE" = "x"; then
> > +-old_CPPFLAGS="$CPPFLAGS"
> > +-for i in $xml2_include_dir /usr/include /usr/local/include 
> > /usr/include/libxml2 /usr/local/include/libxml2 ; do
> > +-CPPFLAGS="$old_CPPFLAGS -I$i"
> > +-AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[#include 
> > ]], [[int a = 
> > 1;]])],[XML2_INCLUDE="-I$i"],[XML2_INCLUDE=
> > +-])
> > +-if test "x$XML2_INCLUDE" != "x"; then
> > +-break;
> > +-fi
> > +-done
> > +-CPPFLAGS="$old_CPPFLAGS $XML2_INCLUDE"
> > ++AC_CHECK_LIB([xml2], [xmlParseFile], [libxml2_found=yes])
> > ++if test "$libxml2_found" = "yes" ; then
> > ++AC_DEFINE(HAVE_LIBXML2,1,[defined when libxml2 is available])
> > ++else
> > ++AC_MSG_WARN([libxml2 not found. Disabling cache.])
> > + fi
> > +
> > + AC_CHECK_HEADERS([libxml/xmlmemory.h])
> > +--
> > +2.34.1
> > +
> > diff --git 
> > a/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch 
> > b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
> > new file mode 100644
> > index 0..d3eb047fe
> > --- /dev/null
> > +++ b/meta-multimedia/recipes-multimedia/spandsp/spandsp/makefile.patch
> > @@ -0,0 +1,161 @@
> > +From: =?UTF-8?q?J=C3=B6rg=20Sommer?= 
> > +Date: Thu, 14 Mar 2024 08:08:13 +0100
> > +Subject: [PATCH 1/2] configure: Use AX_PROG_CC_FOR_BUILD for cross 
> > compiling
> > +MIME-Version: 1.0
> > +Content-Type: text/plain; charset=UTF-8
> > +Content-Transfer-Encoding: 8bit
> > +
> > +The macro AX_PROG_CC_FOR_BUILD [1]