[Buildroot] [PATCH] bctoolbox: fix typos and logic error
Waldemar Brodkorb
wbx at openadk.org
Tue Feb 14 15:39:58 UTC 2017
Hi,
Jörg Krause wrote,
> Hi Waldemar,
>
> On Tue, 2017-02-14 at 10:01 +0100, Waldemar Brodkorb wrote:
> > To get the directory path from the library name you need to
> > use a single filename. Fix typo in variable name.
> >
> > This fixes the ortp autobuild errors which uses the broken pkgconfig
> > file
> > from bctoolbox.
> >
> > Fixes:
> > http://autobuild.buildroot.net/results/37d5625df4be11ccdc063871e9f6
> > e13d5f59fb52
> > http://autobuild.buildroot.net/results/1999c841fae41f860f00747a3623
> > 27cb2857e687
> >
> > Signed-off-by: Waldemar Brodkorb <wbx at openadk.org>
> > ---
> > package/bctoolbox/0001-fix-typo.patch | 18 ++++++++++++++++++
> > 1 file changed, 18 insertions(+)
> > create mode 100644 package/bctoolbox/0001-fix-typo.patch
> >
> > diff --git a/package/bctoolbox/0001-fix-typo.patch
> > b/package/bctoolbox/0001-fix-typo.patch
> > new file mode 100644
> > index 0000000..b94daee
> > --- /dev/null
> > +++ b/package/bctoolbox/0001-fix-typo.patch
> > @@ -0,0 +1,18 @@
> > +Fix a typo in the variable name. Only check path for a single
> > library name.
> > +
> > +Signed-off-by: Waldemar Brodkorb <wbx at openadk.org>
> > +
> > +diff -Nur bctoolbox-0.4.0.orig/CMakeLists.txt bctoolbox-
> > 0.4.0/CMakeLists.txt
> > +--- bctoolbox-0.4.0.orig/CMakeLists.txt 2016-10-06
> > 17:30:41.000000000 +0200
> > ++++ bctoolbox-0.4.0/CMakeLists.txt 2017-02-13
> > 23:30:38.641288032 +0100
> > +@@ -103,8 +103,8 @@
> > + endif()
> > +
> > + if(MBEDTLS_FOUND)
> > +- get_filename_component(mbedtls_library_path
> > "${MBEDTLS_LIBRARIES}" PATH)
> > +- set(LIBS_PRIVATE "${LIBS_PRIVATE}
> > -L${mbedlts_library_path}")
> > ++ get_filename_component(mbedtls_library_path
> > "${MBEDTLS_LIBRARY}" PATH)
> > ++ set(LIBS_PRIVATE "${LIBS_PRIVATE}
> > -L${mbedtls_library_path}")
> > + endif()
> > + if(POLARSSL_FOUND)
> > + get_filename_component(polarssl_library_path
> > "${POLARSSL_LIBRARIES}" PATH)
>
> With this fix applied the bctoolbox pkg-config 'Libs.private:' value is
> set to:
>
> """
> Libs.private: -L/mnt/data/git/buildroot/output/host/usr/arm-buildroot-
> linux-uclibcgnueabi/sysroot/usr/lib32
> """
>
> No mbedtls libraries are included here.
>
> And the flag 'BCTOOLBOX_LIBS' in ortp is set to:
>
> """
> -L/mnt/data/git/buildroot/output/host/usr/arm-buildroot-linux-
> uclibcgnueabi/sysroot/usr/lib -lbctoolbox
> -L/mnt/data/git/buildroot/output/host/usr/ar"\
> "m-buildroot-linux-uclibcgnueabi/sysroot/usr/lib32
> """
>
> From my understanding, all the three mbedtls libraries libmbedcrypto,
> libmbedx509, and libmbedtls should be added to the linker flags and
> therefore to 'Libs.private' in the pkg-config file, right?
I was only looking into the ortp build errors with complete empty -L . But you are right
actually we might need to add -lmbedx509 -lmbedtls -lmbedcrypto
to fix any static linking issues.
I think upstream does add some -l for the polarssl case.
I am not really using this packages, just want to minimize my
architecture related build errors :)
Do you have a chance to suggest a better fix and test with static
linking?
best regards
Waldemar
More information about the buildroot
mailing list