There are some miscellaneous task dependencies, mostly coming from DerivedSources Makefiles which do not declare everything the use or generate.
Created attachment 454298 [details] Patch
(In reply to Elliott Williams from comment #1) > Created attachment 454298 [details] > Patch There are still some dependency errors I'm seeing above WebCore, in WebKit and WebKitLegacy. But this is a large enough set of changes that it's probably time to get it reviewed.
Comment on attachment 454298 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=454298&action=review This is already too long for me to deeply review all parts, but it looks fine. > Tools/ChangeLog:1 > +2022-03-09 Elliott Williams <emw@apple.com> Duplicate ChangeLog.
ChangeLog entry in Tools/ChangeLog contains OOPS!.
Created attachment 454313 [details] Patch
(In reply to Alexey Proskuryakov from comment #3) > > Tools/ChangeLog:1 > > +2022-03-09 Elliott Williams <emw@apple.com> > > Duplicate ChangeLog. Thanks -- fixed in the latest patch, and added your approval. I'll this without additional review.
It's worth noting that testing this patch on EWS broke subsequent EWS builds: Because this patch adds `JSCBuiltins.cpp` as an output file, the EWS builder thinks that the next patch it builds _removes_ JSCBuiltins.cpp, and will helpfully delete it as a stale build product. For example, after building this patch in <https://ews-build.webkit.org/#/builders/61/builds/29660>, the next job <https://ews-build.webkit.org/#/builders/61/builds/29661> failed with: UnifiedSource2.cpp:2:10: fatal error: 'JSCBuiltins.cpp' file not found Higher up in the build log, you see: Remove stale build products /bin/rm -rf /Volumes/Data/worker/macOS-AppleSilicon-Big-Sur-Debug-Build-EWS/build/WebKitBuild/Debug/DerivedSources/JavaScriptCore/JSCBuiltins.cpp I don't think there's anything worth fixing here. One of the goals of my work is to make it harder to forget to declare an output edge like this, requiring fewer bugfix patches like these that confuse EWS.
Committed r291133 (248293@main): <https://commits.webkit.org/248293@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 454313 [details].
<rdar://problem/90121847>