Bug 181117 – [CMake][Win] Visual Studio invokes make_names.pl twice

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: WebCore Misc.

[Fujii Hironori]

[CMake][Win] Visual Studio invokes make_names.pl twice

There are build logs of first time clean build and second time incremental build.
https://gist.github.com/fujii/e2b46c02cae29e57401df181b69e9494

In the first clean build, for example, HTMLNames.cpp was generated twice in WebCoreDerivedSources and WebCore projects

> 22>------ Build started: Project: WebCoreDerivedSources, Configuration: Debug x64 ------
> 
> 22>Generating ../../DerivedSources/WebCore/HTMLNames.cpp, ../../DerivedSources/WebCore/HTMLNames.h, ../../DerivedSources/WebCore/HTMLElementFactory.cpp, ../../DerivedSources/WebCore/HTMLElementFactory.h, ../../DerivedSources/WebCore/JSHTMLElementWrapperFactory.cpp, ../../DerivedSources/WebCore/JSHTMLElementWrapperFactory.h
> 
> 32>------ Build started: Project: WebCore (WebCore\WebCore), Configuration: Debug x64 ------
> 
> 32>Generating ../../DerivedSources/WebCore/HTMLNames.cpp, ../../DerivedSources/WebCore/HTMLNames.h, ../../DerivedSources/WebCore/HTMLElementFactory.cpp, ../../DerivedSources/WebCore/HTMLElementFactory.h, ../../DerivedSources/WebCore/JSHTMLElementWrapperFactory.cpp, ../../DerivedSources/WebCore/JSHTMLElementWrapperFactory.h

Relevant issues:

Bug 177286 – [CMake][Win] EventTargetInterfaces.h are generated twice
Bug 163774 – [CMake][Win] Visual Studio invokes make_settings.pl twice

[Fujii Hironori]

> GENERATE_DOM_NAMES(HTML ${WEBCORE_DIR}/html/HTMLAttributeNames.in ${WEBCORE_DIR}/html/HTMLTagNames.in "")

This command generates three files HTMLNames.cpp, HTMLElementFactory.cpp and JSHTMLElementWrapperFactory.cpp.
But, only HTMLElementFactory.cpp and JSHTMLElementWrapperFactory.cpp are listed in Sources.txt.
Then these files are compiled in WebCore project and HTMLNames.cpp is compiled in WebCoreDerivedSources project.

All files should be listed in Sources.txt.

[Fujii Hironori]

(In reply to Fujii Hironori from comment #1)
> All files should be listed in Sources.txt.

After trying this approach, it turned out there is a problems.
gperf generated source files aren't able to be unified because of enum collision.

[Fujii Hironori]

Created attachment 331178 [details]
WIP patch

[Fujii Hironori]

Created attachment 331321 [details]
WIP patch

* The source files in unified source build of WebCore should be listed in WebCore_SOURCES, not in WebCore_DERIVED_SOURCES.
* Should not add a source dependency from ${WEBCORE_DIR}/css/CSSPrimitiveValueMappings.h to CSSValueKeywords.h, which causes unnecessary build rule in WebCoreForwardingHeaders project.
* Remove source dependency of ${WEBCORE_DIR}/css/CSSParser.cpp because no such file ToT
* Should not add WebCoreJSBuiltinInternals.cpp, WebCoreJSBuiltins.cpp and CSSValueKeywords.cpp to WebCore unified source build because they should be built in WebCoreDerivedSources project

[Fujii Hironori]

Created attachment 331325 [details]
WIP patch

[Fujii Hironori]

I'm going to try another approach in Bug 181664.

[Fujii Hironori]

I have confirmed this bug has been fixed by Bug 181664.

*** This bug has been marked as a duplicate of bug 181664 ***