Bug 198853

Summary: [CMake][Win] CombinedDomains.json is generated twice in JavaScriptCore_CopyPrivateHeaders and JavaScriptCore projects
Product: WebKit Reporter: Fujii Hironori <Hironori.Fujii>
Component: CMakeAssignee: Fujii Hironori <Hironori.Fujii>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, don.olmstead
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
WIP Patch
none
Patch none

Fujii Hironori
Reported 2019-06-14 00:34:45 PDT
[CMake][Win] CombinedDomains.json is generated twice in JavaScriptCore_CopyPrivateHeaders and JavaScriptCore projects Here is the excerpt from build log: > 8>------ Build started: Project: JavaScriptCore_CopyPrivateHeaders, Configuration: Debug x64 ------ > 8>Generating ../../JavaScriptCore/DerivedSources/CombinedDomains.json > 25>------ Build started: Project: JavaScriptCore (JavaScriptCore\JavaScriptCore), Configuration: Debug x64 ------ > 25>Generating ../../JavaScriptCore/DerivedSources/CombinedDomains.json This causes unnecessary re-invoking generate-inspector-protocol-bindings.py in incremental builds because generated InspectorBackendDispatchers.cpp file becomes older than input CombinedDomains.json file.
Attachments
WIP Patch (532 bytes, patch)
2019-06-14 00:51 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Patch (1.80 KB, patch)
2019-06-19 02:15 PDT, Fujii Hironori 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Fujii Hironori
Comment 1 2019-06-14 00:36:39 PDT
Uploaded full build log: https://gist.github.com/fujii/f5b9c0875f50de7f3e1fe244f65fa200
Fujii Hironori
Comment 2 2019-06-14 00:51:02 PDT
Created attachment 372111 [details] WIP Patch The fundamental issue of this bug was fixed in CMake side. https://gitlab.kitware.com/cmake/cmake/issues/16767 However, JavaScriptCore_CopyPrivateHeaders needs to have a direct or indirect dependency of JavaScriptCore for CMake eliminating duplicated custom commands.
Don Olmstead
Comment 3 2019-06-14 10:19:21 PDT
Patch is fine I'm just thinking maybe you should put a comment on why its there. Seems like any WEBKIT_COPY_FILES needs to have a direct dependency if any of the headers contain derived sources.
Fujii Hironori
Comment 4 2019-06-19 01:53:22 PDT
I don't see any other issues.
Fujii Hironori
Comment 5 2019-06-19 02:15:37 PDT
Created attachment 372445 [details] Patch
WebKit Commit Bot
Comment 6 2019-06-19 08:07:37 PDT
Comment on attachment 372445 [details] Patch Clearing flags on attachment: 372445 Committed r246587: <https://trac.webkit.org/changeset/246587>
WebKit Commit Bot
Comment 7 2019-06-19 08:07:39 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.