Bug 58437

Summary: WebKit fails to build with gcc 4.6
Product: WebKit Reporter: Thierry Reding <thierry.reding>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, thierry.reding
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Linux   
Attachments:
Description Flags
Fix build with gcc 4.6.
none
Fix build with gcc 4.6. none

Description Thierry Reding 2011-04-13 06:10:09 PDT 
When the Source/WebCore/dom/make_names.pl script is run, the C preprocessor
is run with the -P option. Apparently, the preprocessor behavior was changed
in gcc 4.6 because running with the -P option makes it eat all empty lines,
which gcc 4.5 doesn't. The make_names.pl script, however, relies on these
empty lines when parsing.

The attached patch removes the -P option when invoking the preprocessor.
Apart from keeping the empty lines, this has the side-effect of outputting
some preprocessor output. However these lines (starting with a #) are
already being ignored by make_names.pl such that the preprocessed files can
be properly parsed.
Comment 1 Thierry Reding 2011-04-13 06:13:57 PDT 
Created attachment 89369 [details]
Fix build with gcc 4.6.
Comment 2 Alexey Proskuryakov 2011-04-13 11:35:17 PDT 
Thank you for reporting and fixing the problem!

For this patch to be landed in trunk, it needs to be prepared in accordance with <http://www.webkit.org/coding/contributing.html>. Specifically, it must have a ChangeLog entry, and it must be marked for review.

For the record, 
       -P  Inhibit generation of linemarkers in the output from the preprocessor.  This might be useful when
           running the preprocessor on something that is not C code, and will be sent to a program which
           might be confused by the linemarkers.
Comment 3 Thierry Reding 2011-04-14 00:14:50 PDT 
Created attachment 89542 [details]
Fix build with gcc 4.6.
Comment 4 Adam Barth 2011-04-17 21:43:12 PDT 
Comment on attachment 89542 [details]
Fix build with gcc 4.6.

Thanks!
Comment 5 WebKit Commit Bot 2011-04-17 23:53:43 PDT 
Comment on attachment 89542 [details]
Fix build with gcc 4.6.

Clearing flags on attachment: 89542

Committed r84123: <http://trac.webkit.org/changeset/84123>
Comment 6 WebKit Commit Bot 2011-04-17 23:53:48 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 7 WebKit Commit Bot 2011-04-18 01:03:33 PDT 
The commit-queue encountered the following flaky tests while processing attachment 89542 [details]:

http/tests/xmlhttprequest/cross-origin-preflight-get.html bug 53976
The commit-queue is continuing to process your patch.
Comment 8 WebKit Commit Bot 2011-04-18 01:30:59 PDT 
The commit-queue encountered the following flaky tests while processing attachment 89542 [details]:

fast/canvas/webgl/gl-teximage.html bug 58766 (authors: kbr@google.com, mihaip@chromium.org, and zmo@google.com)
fast/loader/recursive-before-unload-crash.html bug 50880 (authors: beidson@apple.com and eric@webkit.org)
The commit-queue is continuing to process your patch.