Bug 150119
| Summary: | [WinCairo] Fix incremental build issue revealed by r191017 | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Csaba Osztrogonác <ossy> |
| Component: | New Bugs | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | Normal | CC: | achristensen, ossy, peavo |
| Priority: | P2 | ||
| Version: | Other | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
Csaba Osztrogonác
https://trac.webkit.org/changeset/191017 added a new header file - WTF/wtf/PointerComparison.h
and added includes of this new header file to many cpp files.
But the incremental build failed: https://build.webkit.org/builders/WinCairo%2064-Bit%20Release/builds/50106
c:\users\alex\documents\wincairobot\win-cairo-release\build\source\webcore\rendering\style\FillLayer.cpp(25): fatal error C1083: Cannot open include file: 'wtf/PointerComparison.h': No such file or directory
...
... but the clean build feature is broken on the WinCairo bot, see bug149625 for details.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Alex Christensen
PointerComparison.h should have been added to WTF_HEADERS, and CMake would have been run again, and the header would've been copied. EWS must have been doing a clean build, because it passed. I'm not sure what to do about this.
Csaba Osztrogonác
(In reply to comment #1)
> PointerComparison.h should have been added to WTF_HEADERS, and CMake would
> have been run again, and the header would've been copied. EWS must have
> been doing a clean build, because it passed. I'm not sure what to do about
> this.
Why should we add each header files to the cmake project?
And what is this copying? Is it something WinCairo specific?
Linux ports and Apple Windows port don't need it at all.
Can we make the WinCairo build happier without adding
each header file to the build system?
Csaba Osztrogonác
It seems nobody fixes it.