Bug 18239

Summary: [Gtk] Build breaks if SVG is enabled but SVG Fonts disabled
Product: WebKit Reporter: Marcin Juszkiewicz <openembedded>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: alp, jchaffraix, jmalonzo
Priority: P2 Keywords: Gtk
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
build error log
none
Add SVG_FONTS in SVGAltGlyphElement.idl oliver: review+

Description Marcin Juszkiewicz 2008-03-31 10:40:21 PDT 
Today I tried to upgrade Webkit in our tree to revision newer then 30762 which we used. Build of 31457 rev failed so I tried 31450 and 31400 to check does it happen in older versions too - it was.

On IRC Julien Chaffraix told me that error is due to wrong use of ENABLE(SVG) ENABLE(SVG_FONTS) flags in SVGAltGlyphElement.h and JSVGAltGlyphElement.h header files.

Configure flags used was "--enable-svg --enable-debug=no --enable-icon-database=yes". I tried to cross compile for ARM architecture using Poky Linux buildsystem.
Comment 1 Marcin Juszkiewicz 2008-03-31 10:40:53 PDT 
Created attachment 20238 [details]
build error log
Comment 2 Julien Chaffraix 2008-03-31 11:04:06 PDT 
The issue here is that we try to compile files that need SVG_FONTS even if we do not have specified it.
Switching the platform to Gtk (I thought first that the problem was cross-platform but after a second thought it may not be).
Comment 3 Marcin Juszkiewicz 2008-03-31 11:14:20 PDT 
NOTE: package webkit-gtk-0.1+svnr31467-r4: task do_build: completed

adding "--enable-svg-fonts" helped
Comment 4 Jan Alonzo 2008-04-13 16:09:57 PDT 
Created attachment 20508 [details]
Add SVG_FONTS in SVGAltGlyphElement.idl

The patch adds SVG_FONTS in the Conditional section of SVGAltGlyphElement.idl to prevent the generated source files from being compiled.
Comment 5 Alp Toker 2008-04-13 19:49:03 PDT 
Landed in r31855.