Bug 36751 - [Qt] Windows CE build is broken
Summary: [Qt] Windows CE build is broken
Status: CLOSED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other Other
: P1 Blocker
Assignee: Simon Hausmann
URL:
Keywords: Qt
Depends on:
Blocks: 35784
  Show dependency treegraph
 
Reported: 2010-03-29 03:31 PDT by Simon Hausmann
Modified: 2010-04-20 12:02 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.35 KB, patch)
2010-03-29 03:39 PDT, Simon Hausmann
no flags Details | Formatted Diff | Diff
Patch v2 (2.09 KB, patch)
2010-04-15 11:15 PDT, Jocelyn Turcotte
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Hausmann 2010-03-29 03:31:25 PDT
From https://bugs.webkit.org/show_bug.cgi?id=36590#c5 :

pcre_compile.cpp
c:\qt\wince5\webkit\javascriptcore\wtf\unicode\qt4/UnicodeQt4.h(31) : fatal
error C1083: Cannot open include file: 'stdint.h'
: No such file or directory
pcre_exec.cpp
c:\qt\wince5\webkit\javascriptcore\wtf\unicode\qt4/UnicodeQt4.h(31) : fatal
error C1083: Cannot open include file: 'stdint.h'
: No such file or directory
pcre_tables.cpp
c:\qt\wince5\webkit\javascriptcore\wtf\unicode\qt4/UnicodeQt4.h(31) : fatal
error C1083: Cannot open include file: 'stdint.h'
: No such file or directory
pcre_ucp_searchfuncs.cpp
c:\qt\wince5\webkit\javascriptcore\wtf\unicode\qt4/UnicodeQt4.h(31) : fatal
error C1083: Cannot open include file: 'stdint.h'
: No such file or directory
pcre_xclass.cpp
c:\qt\wince5\webkit\javascriptcore\wtf\unicode\qt4/UnicodeQt4.h(31) : fatal
error C1083: Cannot open include file: 'stdint.h'
: No such file or directory


JavaScriptCore/os-win32 is missing from the include paths.
Comment 1 Simon Hausmann 2010-03-29 03:39:27 PDT
Created attachment 51896 [details]
Patch
Comment 2 Ismail Donmez 2010-03-29 03:41:30 PDT
(In reply to comment #1)
> Created an attachment (id=51896) [details]
> Patch

+
+#    DEFINES += ENABLE_SVG=0 ENABLE_XPATH=0 ENABLE_XBL=0 \
+#               ENABLE_SVG_ANIMATION=0 ENABLE_SVG_USE=0  \
+#               ENABLE_SVG_FOREIGN_OBJECT=0 ENABLE_SVG_AS_IMAGE=0

those seems unrelated.
Comment 3 Ismail Donmez 2010-03-30 07:57:47 PDT
Patch fixes the compilation error for me, thanks.
Comment 4 Eric Seidel (no email) 2010-04-01 15:59:34 PDT
Comment on attachment 51896 [details]
Patch

Can we remove the:
 #    DEFINES += ENABLE_SVG=0 ENABLE_XPATH=0 ENABLE_XBL=0 \
 59 #               ENABLE_SVG_ANIMATION=0 ENABLE_SVG_USE=0  \
 60 #               ENABLE_SVG_FOREIGN_OBJECT=0 ENABLE_SVG_AS_IMAGE=0

The commented out code seems usless.
Comment 5 Jocelyn Turcotte 2010-04-15 11:15:33 PDT
Created attachment 53454 [details]
Patch v2

Upstreaming the patch currently in qt/src/3rdparty/webkit
The same as the previous patch except the commented part has been removed
Comment 6 Simon Hausmann 2010-04-16 16:07:41 PDT
I can't really review a patch with my own name in it, but it looks good to me ;)
Comment 7 Eric Seidel (no email) 2010-04-18 16:27:27 PDT
Comment on attachment 51896 [details]
Patch

Cleared Eric Seidel's review+ from obsolete attachment 51896 [details] so that this bug does not appear in http://webkit.org/pending-commit.
Comment 8 Laszlo Gombos 2010-04-18 21:13:25 PDT
Comment on attachment 53454 [details]
Patch v2

lgtm as well, r+.
Comment 9 WebKit Commit Bot 2010-04-18 21:22:53 PDT
Comment on attachment 53454 [details]
Patch v2

Clearing flags on attachment: 53454

Committed r57802: <http://trac.webkit.org/changeset/57802>
Comment 10 WebKit Commit Bot 2010-04-18 21:22:58 PDT
All reviewed patches have been landed.  Closing bug.
Comment 11 Simon Hausmann 2010-04-20 12:02:34 PDT
Revision r57802 cherry-picked into qtwebkit-2.0 with commit 2ccb40583521dc3a8744555663359671a28aaa2a