trinity-devel@lists.pearsoncomputing.net

Message: previous - next
Month: September 2010

Re: [trinity-devel] #include FT_FREETYPE_H in kdelibs/kdeui/kaction.cpp

From: "Timothy Pearson" <kb9vqf@...>
Date: Sun, 12 Sep 2010 17:02:42 -0500
> On Monday 13 September 2010 00:39:09 Timothy Pearson wrote:
>> > Hello,
>> >
>> > I found that new revision of kdelibs contains these new headers. Are
>> > really
>> > needed there? In any case, i cannot found where FT_FREETYPE_H is set.
>
> [...]
>
>> I don't know why "#include FREETYPE" is present, but the X11 includes
>> are
>> required.  I would personally leave FT_FREETYPE_H in place unless it is
>> causing a build failure under CMake, in which case it can be removed as
>> we
>> are not yet in the feature freeze.
>>
>> Tim
>
> Yes, build failing because FT_FREETYPE_H is not defined. Also, compiling
> fine
> even without X11/freetype inclusions. I can't see in source any
> X11/freetype
> calls.
>
> Actually 3 files have this strange inclusion: kactionclasses.cpp,
> kaction.cpp,
> and kfontcombo.cpp.
>
> --
> Serghei
>

Well, you can exercise your new SVN account and remove the FT_FREETYPE_H
from those files if you would like. ;-)  Just let me know when you have
done so, and verified that it does fix your build issue, so that I can
initiate a full rebuild test of Trinity.

Tim