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

mbedtls 3.x support #11195

Closed
andyli opened this issue Apr 30, 2023 · 2 comments · Fixed by #11646
Closed

mbedtls 3.x support #11195

andyli opened this issue Apr 30, 2023 · 2 comments · Fixed by #11646

Comments

@andyli
Copy link
Member

andyli commented Apr 30, 2023

A few distros pumped their mbedtls 2.x to 3.x (see mbedtls on repology) and it's beginning to affect us - openSUSE neko package build is failing.

Hopefully we can support both 2.x and 3.x, by the use of C preprocessing macros.

@alucryd
Copy link

alucryd commented May 1, 2023

Same thing with archlinux. We made a compat package for the time being (with headers in /usr/include/mbedtls2, and libs in /usr/lib/mbedtls2), but the dune build system does not respect CFLAGS, even though they've been added as per dune's documentation.

(include_subdirs no)

(library
	(name mbedtls)
	(foreign_stubs
		(language c)
		(names mbedtls_stubs)
		(flags -I/usr/include/mbdetls2))
	(c_library_flags (-L/usr/lib/mbedtls2))
	(wrapped false)
)

@kLabz
Copy link
Contributor

kLabz commented Apr 15, 2024

Same thing with archlinux. We made a compat package for the time being (with headers in /usr/include/mbedtls2, and libs in /usr/lib/mbedtls2), but the dune build system does not respect CFLAGS, even though they've been added as per dune's documentation.

(include_subdirs no)

(library
	(name mbedtls)
	(foreign_stubs
		(language c)
		(names mbedtls_stubs)
		(flags -I/usr/include/mbdetls2))
	(c_library_flags (-L/usr/lib/mbedtls2))
	(wrapped false)
)

Thanks! I managed to get it working locally on arch with mbedtls2 package and this libs/mbedtls/dune:

(include_subdirs no)

(library
        (name mbedtls)
        (foreign_stubs
                (language c)
                (names mbedtls_stubs)
                (flags -I/usr/include/mbdetls2 -fPIC))
        (c_library_flags (-L/usr/lib/mbedtls2))
        (wrapped false)
)

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 a pull request may close this issue.

3 participants