No, it doesn't work because on mbedtls version 3.x you will need to look for major version on mbedtls/build_info.h instead of mbedtls/version.h

❯ grep '#define MBEDTLS_VERSION_MAJOR' /usr/include/mbedtls/*.h
/usr/include/mbedtls/build_info.h:#define MBEDTLS_VERSION_MAJOR  3

Making s/version/build_info/ on your patch made it to build fine on debian 13.

On 21/02/26 05:25, Florin Coras via lists.fd.io wrote:
Let’s disable it then for 3.x or greater. Does the new gerrit work?

Florin

On Feb 20, 2026, at 4:12 AM, Renato Botelho do Couto via lists.fd.io 
<[email protected]> wrote:

On 20/02/26 05:04, Florin Coras via lists.fd.io wrote:
Thanks, Renato.
Seems that just removing the header is enough [1]. This is a legacy plugin with 
no tests so that should do for now.
Regards,
Florin
[1] https://gerrit.fd.io/r/c/vpp/+/44983

On debian 13 with mbedtls 3.x installed it doesn't build just by removing the 
bad include, see attached log.

--
Renato Botelho do Couto
Software Engineer
<log.txt>









--
Renato Botelho do Couto
Software Engineer
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#26841): https://lists.fd.io/g/vpp-dev/message/26841
Mute This Topic: https://lists.fd.io/mt/117899261/21656
Group Owner: [email protected]
Unsubscribe: https://lists.fd.io/g/vpp-dev/leave/14379924/21656/631435203/xyzzy 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to