This is happening because “Font.h” is referring to “<WebCore/CoreGraphicsSPI.h>”, which doesn’t always exist on Windows at the time the file is compiled. The CoreGraphicsSPI.h file lives in “WebCore/platform/spi/cg/CoreGraphicsSPI.h”, and can be found with a normal ‘#include “CoreGraphicsSPI.h”. It’s only after a post-build step copies the file to “DerivedSources/ForwardingHeaders/WebCore/CoreGraphicsSPI.h” that the ‘#include <WebCore/CoreGraphicsSPI.h>” form works.
When a non-clean build is performed, this is fine, so EWS probably works properly most of the time. But if something prompts it to clean the build output, the file doesn’t exist and the build fails, improperly blaming the current patch. I’ll talk to Alex about the copy phase of these headers and see if we can get this into proper position earlier. Thanks, -Brent > On Mar 30, 2016, at 9:28 AM, Alexey Proskuryakov <aproskurya...@gmail.com> > wrote: > > > They fail like this: > https://webkit-queues.webkit.org/queue-status/win-ews/bots/ews202https://webkit-queues.webkit.org/results/1069527 > > c:\cygwin\home\buildbot\webkit\source\webcore\platform\graphics\Font.h(57): > fatal error C1083: Cannot open include file: 'WebCore/CoreGraphicsSPI.h': No > such file or directory (compiling source file > C:\cygwin\home\buildbot\WebKit\Source\WebCore\DerivedSources.cpp) > [C:\cygwin\home\buildbot\WebKit\WebKitBuild\Release\Source\WebCore\WebCoreDerivedSources.vcxproj] > > It's quite surprising that the build passes after rolling out a patch, thus > making EWS think that the patch is to blame. > > - Alexey > > > >> 30 марта 2016 г., в 9:20, Brent Fulgham <bfulg...@apple.com> написал(а): >> >> Aside from ews206 being offline for some reason, all bot seem to be running >> without any errors. >> >> Can you point me at a couple of the patches you were looking at? I >> spot-checked a couple in the Review Queue and they seemed to be failing for >> legitimate problems with the patches. >> >> Thanks, >> >> -Brent >> >>> On Mar 30, 2016, at 9:04 AM, Brent Fulgham <bfulg...@apple.com> wrote: >>> >>> Looking now. >>> >>> -Brent >>> >>>> On Mar 30, 2016, at 9:02 AM, Darin Adler <da...@apple.com> wrote: >>>> >>>> Every patch I look at has a red bubble for Windows on EWS. Is someone >>>> planning on fixing this? >>>> >>>> — Darin >>>> _______________________________________________ >>>> webkit-dev mailing list >>>> webkit-dev@lists.webkit.org >>>> https://lists.webkit.org/mailman/listinfo/webkit-dev >>> >>> _______________________________________________ >>> webkit-dev mailing list >>> webkit-dev@lists.webkit.org >>> https://lists.webkit.org/mailman/listinfo/webkit-dev >> >> _______________________________________________ >> webkit-dev mailing list >> webkit-dev@lists.webkit.org >> https://lists.webkit.org/mailman/listinfo/webkit-dev > > _______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org https://lists.webkit.org/mailman/listinfo/webkit-dev
