Bug 246760 - [CMake] All post-commit buildbots should build with --no-fatal-warnings
Summary: [CMake] All post-commit buildbots should build with --no-fatal-warnings
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CMake (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-10-19 12:33 PDT by Michael Catanzaro
Modified: 2022-10-25 06:38 PDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2022-10-19 12:33:43 PDT
I intend to enable DEVELOPER_MODE_FATAL_WARNINGS by default to finally fix bug #155047, since I've been spending way too much time fighting build warnings instead of doing more interesting work, and really need the EWS bots to enforce fatal warnings. But we never want to lose test results to a silly build warning: that would be brutal punishment for somebody introducing a new warning. So for this to work, we need post-commit buildbots (but not EWS) to use build-webkit --no-fatal-warnings. Post-commit bots that don't use build-webkit should use the -DDEVELOPER_MODE_FATAL_WARNINGS=OFF CMake option. 

I'm not sure the best way to do this. We could add additional arguments to every bot in Tools/CISupport/build-webkit-org/config.json, but that would need to be repeated for every single bot. Maybe there is another way? Don't want to change build-webkit because fatal warnings should be default in developer mode. (The name of the option DEVELOPER_MODE_FATAL_WARNINGS doesn't make much sense otherwise.)
Comment 1 Michael Catanzaro 2022-10-24 09:28:08 PDT
Pull request: https://github.com/WebKit/WebKit/pull/5710
Comment 2 EWS 2022-10-25 06:38:04 PDT
Committed 255954@main (e868bfe3d845): <https://commits.webkit.org/255954@main>

Reviewed commits have been landed. Closing PR #5710 and removing active labels.