Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 407825 - dev-libs/tinyxml[stl] could/should add '#define TIXML_USE_STL 1' to tinyxml.h
Summary: dev-libs/tinyxml[stl] could/should add '#define TIXML_USE_STL 1' to tinyxml.h
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All All
: Normal normal (vote)
Assignee: Bernard Cafarelli
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-03-11 17:26 UTC by Karol Herbst
Modified: 2012-03-14 23:07 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Karol Herbst 2012-03-11 17:26:11 UTC
while building desurium (own fork: https://github.com/karolherbst/Desurium ) with system installed tinyxml (USE flags: -debug -doc -static-libs +stl) I got undefined reference to `TiXmlString::nullrep_`.

The unforked version of desurium will build tinyxml itself

Reproducible: Always

Steps to Reproduce:
1. install tinyxml with USE="stl"
2. build desurium (own fork)
Actual Results:  
build failure:
../../shared/usercore/libusercore.so: undefined reference to `TiXmlString::nullrep_

Expected Results:  
no build failure

it works with USE="-stl"
Comment 1 Samuli Suominen (RETIRED) gentoo-dev 2012-03-11 17:31:29 UTC
You need to compile your project with -DTIXML_USE_STL or #define it in the headers if you want STL
Comment 2 Karol Herbst 2012-03-11 19:21:47 UTC
I don't need stl functionality in any way. But tinyxml is built with stl by default, which causes this undefined reference.

So I would have to know whether tinyxml is built with stl or not in the project. But since I want to use system libraries I would like the system to put that into the header file like in archlinux, when built with stl.

Archlinux is using the following patch to fix that:
https://projects.archlinux.org/svntogit/community.git/tree/trunk/tinyxml-2.5.3-stl.patch?h=packages/tinyxml
Comment 3 Samuli Suominen (RETIRED) gentoo-dev 2012-03-12 07:54:34 UTC
ideally tinyxml should ship pkg-config file, but that would be deviating from upstream even futher... moving to maintainer for review
Comment 4 Bernard Cafarelli gentoo-dev 2012-03-14 23:07:22 UTC
I quite like the idea (even if a clean pkg-config file would be cleaner indeed), this will allow packages depending on tinyxml[stl] to drop the append-cxxflags (filezilla and gource at least)

Added in tinyxml-2.6.2-r2, thanks for the report and link