[Buildroot] [PATCH 1/1] package/pugixml: drop BR2_PACKAGE_PUGIXML_HEADER_ONLY

Thomas Petazzoni thomas.petazzoni at bootlin.com
Sun Jan 31 22:30:21 UTC 2021


On Thu, 28 Jan 2021 18:29:35 +0100
Fabrice Fontaine <fontaine.fabrice at gmail.com> wrote:

> Drop BR2_PACKAGE_PUGIXML_HEADER_ONLY which raises a build failure with
> gerbera (the only package that is using pugixml). Indeed, there is two
> issues:
>  - gerbera is not aware that it must define PUGIXML_HEADER_ONLY
>  - pugixml.cpp is not available in the staging directory so gerbera
>    can't use it
> 
> It should be noted that pugixml continues to install a (non working)
> library even when PUGIXML_HEADER_ONLY is enabled resulting in a build
> failure at linking stage instead of configure step:
> 
> [100%] Linking CXX executable gerbera
> /home/buildroot/autobuild/run/instance-2/output-1/host/opt/ext-toolchain/bin/../lib/gcc/arm-buildroot-linux-musleabihf/8.3.0/../../../../arm-buildroot-linux-musleabihf/bin/ld: liblibgerbera.a(config_manager.cc.o): in function `ConfigManager::getElement(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >) const':
> config_manager.cc:(.text+0x4d8): undefined reference to `pugi::xml_document::document_element() const'

This looks like an upstream bug in pugixml, isn't it ? We're using
pugixml 1.10, but a 1.11 has been released since then. Have you tried
that ? Did you ask upstream pugixml about this issue ?

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com



More information about the buildroot mailing list