Bug 220251 - [WinCairo][CMake][MSBuild][ANGLE] entry_points_*.h was generated twice both in LibGLESv2EntryPointsHeaders and GLESv2 projects
Summary: [WinCairo][CMake][MSBuild][ANGLE] entry_points_*.h was generated twice both i...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: ANGLE (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Fujii Hironori
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-01-04 12:36 PST by Fujii Hironori
Modified: 2021-01-04 15:19 PST (History)
14 users (show)

See Also:


Attachments
WinCairo build log (193.69 KB, text/plain)
2021-01-04 12:37 PST, Fujii Hironori
no flags Details
Patch (2.46 KB, patch)
2021-01-04 13:19 PST, Fujii Hironori
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fujii Hironori 2021-01-04 12:36:04 PST
WinCairo CMake+MSBuild builds report the following error in clean build.

27>C:\home\webkit\gb\WebKitBuild\Release\ANGLE\Headers\ANGLE/entry_points_egl.h(13,10): fatal error C1083: Cannot open include file: 'export.h': No such file or directory (compiling source file C:\home\webkit\gb\Source\WebCore\platform\graphics\texmap\ANGLEContext.cpp)

This issue was reported by Microsoft Playwright team. https://webkit.slack.com/archives/CU84Q46JZ/p1609781605032600

CMake+Ninja can compile.
Comment 1 Fujii Hironori 2021-01-04 12:37:06 PST
Created attachment 416959 [details]
WinCairo build log

entry_points_egl.h was generated twice both in LibGLESv2EntryPointsHeaders and GLESv2 projects.

10>------ Build started: Project: LibGLESv2EntryPointsHeaders, Configuration: Debug x64 ------
10>Generating ../../../ANGLE/Headers/ANGLE/entry_points_egl.h

16>------ Build started: Project: ANGLE-webgl-headers, Configuration: Debug x64 ------
16>Postprocessed ANGLE header entry_points_egl.h

23>------ Build started: Project: GLESv2, Configuration: Debug x64 ------
23>Generating ../../../ANGLE/Headers/ANGLE/entry_points_egl.h
Comment 2 Fujii Hironori 2021-01-04 12:42:33 PST
The fundamental issue of this bug was fixed in CMake v3.12.
https://gitlab.kitware.com/cmake/cmake/-/commit/5a6c6292898fe238f3a5105133b8904209fbedaf
https://gitlab.kitware.com/cmake/cmake/issues/16767

See also:
  Bug 213226 – [CMake][Visual Studio] CSSPropertyNames.h is generated twice in WebCore.vcxproj and WebCore_CopyPrivateHeaders.vcxproj
  Bug 213225 – [CMake][Visual Studio] CombinedDomains.json is generated twice in JavaScriptCore.vcxproj and InspectorBackendCommands.vcxproj
  Bug 198853 – [CMake][Win] CombinedDomains.json is generated twice in JavaScriptCore_CopyPrivateHeaders and JavaScriptCore projects
Comment 3 Fujii Hironori 2021-01-04 13:19:59 PST
Created attachment 416962 [details]
Patch
Comment 4 EWS Watchlist 2021-01-04 13:20:41 PST
Note that there are important steps to take when updating ANGLE. See https://trac.webkit.org/wiki/UpdatingANGLE
Comment 5 Kenneth Russell 2021-01-04 13:57:27 PST
Comment on attachment 416962 [details]
Patch

Looks good to me. r+
Comment 6 Yusuke Suzuki 2021-01-04 15:09:31 PST
Comment on attachment 416962 [details]
Patch

Nice!
Comment 7 EWS 2021-01-04 15:18:16 PST
Committed r271139: <https://trac.webkit.org/changeset/271139>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 416962 [details].
Comment 8 Radar WebKit Bug Importer 2021-01-04 15:19:14 PST
<rdar://problem/72797771>