See description in bug 91105.
Good idea. My feeling is that we should do this in WebCore/DerivedSources.pri and call bison/flex directly, i.e. avoid the generate_parser.sh script. The main reason for avoiding the scripts would be that we don't require bash for the build of the Qt port on Windows.
(In reply to comment #1) > Good idea. My feeling is that we should do this in WebCore/DerivedSources.pri and call bison/flex directly, i.e. avoid the generate_parser.sh script. That makes sense to me, too. > The main reason for avoiding the scripts would be that we don't require bash for the build of the Qt port on Windows. That's really good to know. Thanks, Simon.
Created attachment 152720 [details] Patch
I think this patch could be applied even today, we just won't compile the generated sources that exist in Thirdparty/Angle. I noticed that the "new processor" files don't seem to be currently used though.
(In reply to comment #4) > I think this patch could be applied even today, we just won't compile the generated sources that exist in Thirdparty/Angle. Thanks for the patch, Simon! Just a heads up- we landed a big ANGLE update to r1170 today in bug 89039, so you'll have to make a (hopefully small) change to your patch. The following files were renamed in the new version of ANGLE: ANGLE/src/compiler/preprocessor/new/pp.l -> Tokenizer.l ANGLE/src/compiler/preprocessor/new/pp.y -> ExpressionParser.y You'll need to remove Tokenizer.cpp and ExpressionParser.cpp from WebCore/Target.pri as well, since you'll be generating them now. > I noticed that the "new processor" files don't seem to be currently used though. I think those files are now required for the new version of ANGLE to build. However, the new preprocessor isn't enabled in the current WebKit ANGLE source. Next time we update ANGLE though, it will be using the the new preprocessor.
Created attachment 152950 [details] Patch
Comment on attachment 152950 [details] Patch rs=me
Committed r122947: <http://trac.webkit.org/changeset/122947>