inkscape fails with FT221
I've just tried to install Inkscape 0.43 on my gentoo box and it (or one of its components) seems to use the FT internal headers:
libnrtype/FontInstance.cpp:29:40: freetype/internal/tttypes.h: No such file or directory libnrtype/FontInstance.cpp:30:41: freetype/internal/ftstream.h: No such file or directory
Has anybody else come across this problem or is my box in a state of confusion?
On Wed, May 17, 2006 at 02:28:00PM +0200, David Somers wrote:
I've just tried to install Inkscape 0.43 on my gentoo box and it (or one of its components) seems to use the FT internal headers:
libnrtype/FontInstance.cpp:29:40: freetype/internal/tttypes.h: No such file or directory libnrtype/FontInstance.cpp:30:41: freetype/internal/ftstream.h: No such file or directory
Has anybody else come across this problem or is my box in a state of confusion?
I think you need to install the freetype development package. It should be called libfreetype-dev, -devel or similar.
HTH,
Wolfi
On Wednesday 17 May 2006 14:40, Wolfram Quester wrote:
I think you need to install the freetype development package. It should be called libfreetype-dev, -devel or similar.
Nope.
It appears that libnrtype/FontInstance.cpp wants to pull in the freetype internal headers (and being *internal* they're not really meant to be used, which is why they're called internal).
See http://www.freetype.org/freetype2/freetype-2.2.0.html for more info on this issue.
Hello David,
David Somers a écrit :
I've just tried to install Inkscape 0.43 on my gentoo box and it (or one of its components) seems to use the FT internal headers:
libnrtype/FontInstance.cpp:29:40: freetype/internal/tttypes.h: No such file or directory libnrtype/FontInstance.cpp:30:41: freetype/internal/ftstream.h: No such file or directory
Has anybody else come across this problem or is my box in a state of confusion?
No, it's just that Inkscape is one of the rogue clients, though this was undetected until you tried recompiling it.
Here's a patch that fixes the issue. Ironically, the code doesn't really need the internal headers to compile. The patch also prevents a potential segmentation fault.
Hope it will be accepted for 0.44 soon. Werner, could you add the patch to our public page ?
Regards,
- David Turner - The FreeType Project (www.freetype.org)
On Thu, 18 May 2006, David Turner wrote:
David Somers wrote:
I've just tried to install Inkscape 0.43 on my gentoo box and it (or one of its components) seems to use the FT internal headers:
libnrtype/FontInstance.cpp:29:40: freetype/internal/tttypes.h: No such file or directory libnrtype/FontInstance.cpp:30:41: freetype/internal/ftstream.h: No such file or directory
Here's a patch that fixes the issue. Ironically, the code doesn't really need the internal headers to compile. The patch also prevents a potential segmentation fault.
Hope it will be accepted for 0.44 soon.
Thanks David, the patch has been commited to Inkscape SVN and will be part of 0.44.
Cheers, Michael
Hi David!
your patch to fix inkscape and FT221 contains the following hunk:
@@ -474,12 +474,12 @@
theFace=pango_ft2_font_get_face(pFont);
if ( theFace->units_per_EM == 0 ) return false; // bitmap font
if ( pFont == NULL ) return false;
if ( theFace == NULL ) return false;
- if ( !FT_IS_SCALABLE(theFace) ) return false; // bitmap font
- TT_HoriHeader *hhea = (TT_HoriHeader*)FT_Get_Sfnt_Table(theFace, ft_sfnt_hhea); if (hhea == NULL) return false; run = hhea->caret_Slope_Run;
In this file there are two more occurences of "theFace->units_per_EM == 0" to test if the font is a bitmap font. Should these be replaced by "FT_IS_SCALABLE" too?
Thanks,
Wolfi
Hello,
Wolfram Quester a écrit :
In this file there are two more occurences of "theFace->units_per_EM == 0" to test if the font is a bitmap font. Should these be replaced by "FT_IS_SCALABLE" too?
Yes definitely, the 'units_per_EM == 0' test shouldn't be used to determine wether a font is scalable or not.
Regards,
- David
Thanks,
Wolfi
Freetype-devel mailing list Freetype-devel@...448... http://lists.nongnu.org/mailman/listinfo/freetype-devel
*********************************************************************************** Information contained in this email message is confidential and may be privileged, and is intended only for use of the individual or entity named above. If the reader of this message is not the intended recipient, or the employee or agent responsible to deliver it to the intended recipient, you are hereby notified that any dissemination, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please immediately notify the postmaster@...1309... and destroy the original message. ***********************************************************************************
In this file there are two more occurences of "theFace->units_per_EM == 0" to test if the font is a bitmap font. Should these be replaced by "FT_IS_SCALABLE" too?
Yes definitely, the 'units_per_EM == 0' test shouldn't be used to determine wether a font is scalable or not.
David, please send me an updated patch against 0.43 so that I can fix it on the patches web page.
Werner
participants (5)
-
David Somers -
David Turner -
Michael Wybrow -
Werner LEMBERG -
Wolfram Quester