diff mbox series

[meta-oe,19/33] libyang: upgrade 3.13.6 -> 4.2.2

Message ID 20260225093727.1284-19-wangmy@fujitsu.com
State New
Headers show
Series [meta-oe,01/33] abseil-cpp: upgrade 20260107.0 -> 20260107.1 | expand

Commit Message

Mingyu Wang (Fujitsu) Feb. 25, 2026, 9:37 a.m. UTC
From: Wang Mingyu <wangmy@fujitsu.com>

Changelog:
==========
- new SO version 4
  all the important changes mentioned in the transition manual
- major LYB binary format optimizations
- user-ordered lists diff fixes
- validation optimizations
- lots of other bugfixes and improvements

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
---
 .../libyang/{libyang_3.13.6.bb => libyang_4.2.2.bb}             | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta-oe/recipes-extended/libyang/{libyang_3.13.6.bb => libyang_4.2.2.bb} (96%)

Comments

Khem Raj Feb. 26, 2026, 4:32 a.m. UTC | #1
this breaks frr. Seems frr is still needing v2 APIs but with this version
it has either changed or removed these APIs. Please take a look

| lib/northbound.c: In function 'nb_config_diff':
| lib/northbound.c:641:36: error: 'LYD_PRINT_WITHSIBLINGS' undeclared
(first use in this function); did you mean 'LYD_PRINT_SIBLINGS'?
|   641 |                                    LYD_PRINT_WITHSIBLINGS |
LYD_PRINT_WD_ALL)) {
|       |                                    ^~~~~~~~~~~~~~~~~~~~~~
|       |                                    LYD_PRINT_SIBLINGS
| lib/northbound.c:641:36: note: each undeclared identifier is reported
only once for each function it appears in
| lib/northbound.c: In function 'nb_cb_operation_is_valid':
| lib/northbound.c:2302:29: error: invalid operands to binary || (have
'int' and 'struct lysc_value')
|  2302 |                             || sleaf->dflt)
|       |                             ^~ ~~~~~~~~~~~
|       |                                     |
|       |                                     struct lysc_value



On Wed, Feb 25, 2026 at 1:38 AM wangmy via lists.openembedded.org <wangmy=
fujitsu.com@lists.openembedded.org> wrote:

> From: Wang Mingyu <wangmy@fujitsu.com>
>
> Changelog:
> ==========
> - new SO version 4
>   all the important changes mentioned in the transition manual
> - major LYB binary format optimizations
> - user-ordered lists diff fixes
> - validation optimizations
> - lots of other bugfixes and improvements
>
> Signed-off-by: Wang Mingyu <wangmy@fujitsu.com>
> ---
>  .../libyang/{libyang_3.13.6.bb => libyang_4.2.2.bb}             | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>  rename meta-oe/recipes-extended/libyang/{libyang_3.13.6.bb =>
> libyang_4.2.2.bb} (96%)
>
> diff --git a/meta-oe/recipes-extended/libyang/libyang_3.13.6.bb
> b/meta-oe/recipes-extended/libyang/libyang_4.2.2.bb
> similarity index 96%
> rename from meta-oe/recipes-extended/libyang/libyang_3.13.6.bb
> rename to meta-oe/recipes-extended/libyang/libyang_4.2.2.bb
> index 822c30ba24..4b2ceaaa17 100644
> --- a/meta-oe/recipes-extended/libyang/libyang_3.13.6.bb
> +++ b/meta-oe/recipes-extended/libyang/libyang_4.2.2.bb
> @@ -6,7 +6,7 @@ LICENSE = "BSD-3-Clause"
>
>  LIC_FILES_CHKSUM = "file://LICENSE;md5=b69fd3b2815bbf1cef5c97f0eee2519a"
>
> -SRCREV = "c2ddd01b9b810a30d6a7d6749a3bc9adeb7b01fb"
> +SRCREV = "3d07c3a71534a580c3960907da17568eff7e5c64"
>
>  SRC_URI = "git://
> github.com/CESNET/libyang.git;branch=master;protocol=https;tag=v${PV}
> <http://github.com/CESNET/libyang.git;branch=master;protocol=https;tag=v$%7BPV%7D>
> \
>             file://0001-test_context-skip-test-case-test_searchdirs.patch \
> --
> 2.43.0
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#124631):
> https://lists.openembedded.org/g/openembedded-devel/message/124631
> Mute This Topic: https://lists.openembedded.org/mt/117991961/1997914
> Group Owner: openembedded-devel+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [
> raj.khem@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>
>
Mingyu Wang (Fujitsu) Feb. 26, 2026, 9:31 a.m. UTC | #2
The root cause is that the newer version of libyang has changed or removed certain v2 APIs that frr still depends on, specifically the LYD_PRINT_WITHSIBLINGSflag and the structure used for default values. As noted, this is a compatibility break.
I have also checked the upstream communities for both libyang and frr. There is no official fix or patch available yet to resolve this API incompatibility.
The upgrade should be postponed until the upstream frr project has been adapted to work with the new libyang APIs, or until a compatible patch is provided.

 --
Best Regards
---------------------------------------------------
Wang Mingyu
FUJITSU NANJING SOFTWARE TECHNOLOGY CO., LTD. (FNST)
No.6 Wenzhu Road, Nanjing, 210012, China
TEL:+86+25-86630566--8568
COINS: 79988548
FAX: +86+25-83317685
MAIL: wangmy@fujitsu.com<mailto:wangmy@fujitsu.com>

From: Khem Raj <raj.khem@gmail.com>
Sent: Thursday, February 26, 2026 12:32 PM
To: Wang, Mingyu/王 鸣瑜 <wangmy@fujitsu.com>
Cc: openembedded-devel@lists.openembedded.org
Subject: Re: [oe] [meta-oe] [PATCH 19/33] libyang: upgrade 3.13.6 -> 4.2.2

this breaks frr. Seems frr is still needing v2 APIs but with this version it has either changed or removed these APIs. Please take a look

| lib/northbound.c: In function 'nb_config_diff':
| lib/northbound.c:641:36: error: 'LYD_PRINT_WITHSIBLINGS' undeclared (first use in this function); did you mean 'LYD_PRINT_SIBLINGS'?
|   641 |                                    LYD_PRINT_WITHSIBLINGS | LYD_PRINT_WD_ALL)) {
|       |                                    ^~~~~~~~~~~~~~~~~~~~~~
|       |                                    LYD_PRINT_SIBLINGS
| lib/northbound.c:641:36: note: each undeclared identifier is reported only once for each function it appears in
| lib/northbound.c: In function 'nb_cb_operation_is_valid':
| lib/northbound.c:2302:29: error: invalid operands to binary || (have 'int' and 'struct lysc_value')
|  2302 |                             || sleaf->dflt)
|       |                             ^~ ~~~~~~~~~~~
|       |                                     |
|       |                                     struct lysc_value



On Wed, Feb 25, 2026 at 1:38 AM wangmy via lists.openembedded.org<http://lists.openembedded.org> <wangmy=fujitsu.com@lists.openembedded.org<mailto:fujitsu.com@lists.openembedded.org>> wrote:
From: Wang Mingyu <wangmy@fujitsu.com<mailto:wangmy@fujitsu.com>>

Changelog:
==========
- new SO version 4
  all the important changes mentioned in the transition manual
- major LYB binary format optimizations
- user-ordered lists diff fixes
- validation optimizations
- lots of other bugfixes and improvements

Signed-off-by: Wang Mingyu <wangmy@fujitsu.com<mailto:wangmy@fujitsu.com>>
---
 .../libyang/{libyang_3.13.6.bb<http://libyang_3.13.6.bb> => libyang_4.2.2.bb<http://libyang_4.2.2.bb>}             | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 rename meta-oe/recipes-extended/libyang/{libyang_3.13.6.bb<http://libyang_3.13.6.bb> => libyang_4.2.2.bb<http://libyang_4.2.2.bb>} (96%)

diff --git a/meta-oe/recipes-extended/libyang/libyang_3.13.6.bb<http://libyang_3.13.6.bb> b/meta-oe/recipes-extended/libyang/libyang_4.2.2.bb<http://libyang_4.2.2.bb>
similarity index 96%
rename from meta-oe/recipes-extended/libyang/libyang_3.13.6.bb<http://libyang_3.13.6.bb>
rename to meta-oe/recipes-extended/libyang/libyang_4.2.2.bb<http://libyang_4.2.2.bb>
index 822c30ba24..4b2ceaaa17 100644
--- a/meta-oe/recipes-extended/libyang/libyang_3.13.6.bb<http://libyang_3.13.6.bb>
+++ b/meta-oe/recipes-extended/libyang/libyang_4.2.2.bb<http://libyang_4.2.2.bb>
@@ -6,7 +6,7 @@ LICENSE = "BSD-3-Clause"

 LIC_FILES_CHKSUM = "file://LICENSE;md5=b69fd3b2815bbf1cef5c97f0eee2519a"

-SRCREV = "c2ddd01b9b810a30d6a7d6749a3bc9adeb7b01fb"
+SRCREV = "3d07c3a71534a580c3960907da17568eff7e5c64"

 SRC_URI = "git://github.com/CESNET/libyang.git;branch=master;protocol=https;tag=v${PV}<http://github.com/CESNET/libyang.git;branch=master;protocol=https;tag=v$%7BPV%7D> \
            file://0001-test_context-skip-test-case-test_searchdirs.patch \
--
2.43.0


-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#124631): https://lists.openembedded.org/g/openembedded-devel/message/124631
Mute This Topic: https://lists.openembedded.org/mt/117991961/1997914
Group Owner: openembedded-devel+owner@lists.openembedded.org<mailto:openembedded-devel%2Bowner@lists.openembedded.org>
Unsubscribe: https://lists.openembedded.org/g/openembedded-devel/unsub [raj.khem@gmail.com<mailto:raj.khem@gmail.com>]
-=-=-=-=-=-=-=-=-=-=-=-
diff mbox series

Patch

diff --git a/meta-oe/recipes-extended/libyang/libyang_3.13.6.bb b/meta-oe/recipes-extended/libyang/libyang_4.2.2.bb
similarity index 96%
rename from meta-oe/recipes-extended/libyang/libyang_3.13.6.bb
rename to meta-oe/recipes-extended/libyang/libyang_4.2.2.bb
index 822c30ba24..4b2ceaaa17 100644
--- a/meta-oe/recipes-extended/libyang/libyang_3.13.6.bb
+++ b/meta-oe/recipes-extended/libyang/libyang_4.2.2.bb
@@ -6,7 +6,7 @@  LICENSE = "BSD-3-Clause"
 
 LIC_FILES_CHKSUM = "file://LICENSE;md5=b69fd3b2815bbf1cef5c97f0eee2519a"
 
-SRCREV = "c2ddd01b9b810a30d6a7d6749a3bc9adeb7b01fb"
+SRCREV = "3d07c3a71534a580c3960907da17568eff7e5c64"
 
 SRC_URI = "git://github.com/CESNET/libyang.git;branch=master;protocol=https;tag=v${PV} \
            file://0001-test_context-skip-test-case-test_searchdirs.patch \