trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: August 2013

Re: [trinity-devel] tdelibs FTBFS

From: "Darrell Anderson" <darrella@...>
Date: Fri, 16 Aug 2013 13:54:12 -0500
>Sorry, I'm a little confused, so, slackware doesn't have libbfq at
>all or just demangle.h is missing?

I don't know about "libbfq," but I suspect you meant to type 
"libbfd." :-) Yes, libbfd is part of Slackware binutils.

As I shared previously, demangle.h is not part of Slackware 14.0 or 
previous releases. The next official release, 14.1, WILL contain 
demangle.h in the binutils package.

For my own personal usage, I rebuilt binutils for Slackware 14.0 to 
include demangle.h. Therefore, technically, I no longer need the 
tdelibs patch.

I nonetheless can temporarily rename or delete demangle.h to 
simulate the non-existence of that file and to test any such patch.

The tdelibs patch remains necessary because Slackware is not alone 
in excluding demangle.h from binutils. Search the dev mail list 
archives for a previous discussion about demangle.h and affected 
distros.

When demangle.h is not installed, the recent tdeioslave backtrace 
support patch in tdelibs will not compile.

The tdelibs patch, and the predessor patch for tdebase, only allows 
compiling the modules when demangle.h is not installed in the build 
environment. The patches have nothing to do with the actual 
bactrace support.

Those folks who compile without those patches merely obtain the 
ability to compile without demangle.h. When compiled in that 
manner, those users do not have any ability to produce backtraces 
from the affected code. Only those folks who compile with 
demangle.h installed have the ability to produce backtraces from 
the affected code.

>Anyway in a day or two I'll came with a normal patch to move all
>that to
>kdebug.cpp and provide WITH_LIBBFD for optional linkage... + some
>additional backtrace features.

I don't think WITH_LIBBFD is necessary. Only an option 
WITH_DEMANGLE is necessary, or, in my preliminary draft patch, 
WITH_TDEIOSLAVE_BACKTRACE. I derived my compile option name from 
the previous tdebase patch, which uses WITH_KDESKTOP_LOCK_BACKTRACE.

The whole purpose of this discussion is only to patch tdelibs to 
compile when demangle.h is not installed. Nothing more. Seems the 
proposed changes to my preliminary draft patch from Slavek are all 
that is needed. :-)

Darrell