make check fails

Robby Stephenson robby at periapsis.org
Tue Mar 20 23:04:00 MDT 2007


On Tuesday 20 March 2007, Jens Seidel wrote:
> Right. The problem is that the tests evaluate the config settings from
> ~/.kde/share/config/tellicorc, in this case "Auto Capitalization=false".
>
> The conditional
> if(Config::autoCapitalization())
> in src/field.cpp:217 should not depend on user config settings.
>
> The solution is probably to create a default configuration in the test
> code so that no new config object is created.

Right, of course. I forgot about that, thanks for pointing it out.

> > > PS: AM_CPPFLAGS= is preferred over INCLUDES= which is deprecated.
> > > See autoconf.pdf, page 68.
> >
> > How far backwards compatible is it? I don't want to break that just for
> > the heck of it.
>
> I'm very confident that even 10 year old versions of automake support it.

After a search, it looks like you're right. They equivalence goes back to at 
least 2003, so there aren't probably any drawbacks. I'll go ahead with the 
changes.

> The long term solution is probably a switch to
> cmake anyway, as the rest of KDE. I don't have any experience with it.

Yeah, along with the port to KDE4, that'll come next.

Robby



More information about the tellico-users mailing list