Bug 115958

Summary: Make it a build error to put invalid C++ symbol names in WebCore.exp.in
Product: WebKit Reporter: Anders Carlsson <andersca>
Component: New BugsAssignee: Anders Carlsson <andersca>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, bfulgham, kling
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch kling: review+

Description Anders Carlsson 2013-05-11 08:46:47 PDT
Make it a build error to put invalid C++ symbol names in WebCore.exp.in
Comment 1 Anders Carlsson 2013-05-11 08:48:49 PDT
Created attachment 201459 [details]
Patch
Comment 2 Andreas Kling 2013-05-11 08:59:07 PDT
Comment on attachment 201459 [details]
Patch

Cool idea.
Comment 3 Anders Carlsson 2013-05-11 09:09:11 PDT
Committed r149932: <http://trac.webkit.org/changeset/149932>
Comment 4 Brent Fulgham 2013-05-11 22:48:16 PDT
I didn't think that <cxxabi.h> existed on Windows. At least, my local build now fails.  I wonder why the build bots are happy?
Comment 5 Brent Fulgham 2013-05-11 23:07:56 PDT
Yeah, looking at other cases of <cxxabi.h> use on the web, it appears to be something Microsoft has not yet implemented.  We should probably enclose this in an OS(WINDOWS) guard or something.

Note that the VS2010 version of this generator was not touched by this change. Maybe the right fix here is to copy an old version of the generator script to the vcproj folder and use it exclusively for VS2005 builds.

Either way, I'm guessing the Windows bots will start failing soon when they do a full clean build, or something touches the symbol file kicking of a rebuild of the WebKitExportGenerator project.  For better or for worse, the actual script that generates the source file is not 'seen' by the VS dependency checker, so it does not trigger a build when changed.  That's probably why the builds are still running.
Comment 6 Brent Fulgham 2013-05-11 23:17:54 PDT
I committed a change to resolve the problem on my side:

Committed r149951: <http://trac.webkit.org/changeset/149951>

Please double-check I did not hurt anything in your new functionality.

Thanks!
Comment 7 Anders Carlsson 2013-05-12 13:38:02 PDT
(In reply to comment #6)
> I committed a change to resolve the problem on my side:
> 
> Committed r149951: <http://trac.webkit.org/changeset/149951>
> 
> Please double-check I did not hurt anything in your new functionality.
> 
> Thanks!

Sorry about that, I didn't know that ExportFileGenerator runs on Windows! (I don't think  it needs to though).