Bug 249791 - [WPE] Temporarily disable ENABLE_COG by default
Summary: [WPE] Temporarily disable ENABLE_COG by default
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: WPE WebKit (show other bugs)
Version: WebKit Nightly Build
Hardware: PC Linux
: P2 Normal
Assignee: Michael Catanzaro
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-12-22 11:30 PST by Michael Catanzaro
Modified: 2023-01-05 13:58 PST (History)
2 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-12-22 11:30:52 PST
It's time to start making breaking changes for the WPE 2.0 API version, but we cannot actually do this currently because our ENABLE_COG option depends on an external checkout of cog that already depends on WPE 2.0. Removing API that cog uses results in WPE port build failure. But we also cannot plausibly change cog to use new APIs that do not yet exist in WebKit because we cannot commit them because we have not yet updated cog. Aaah, it's circular!

I see two workable options:

 1. WebKit must depend on side branches of cog, where cog is modified to account for WebKit API changes that have not yet landed in WebKit
 2. Simply turn off ENABLE_COG for now, and reenable right before releasing 2.39.90 when the API stabilizes

My preference is option 2 because that would be easier for WebKit development, although it risks that we forget to keep cog in good shape. We'll want to remember to turn cog on again in February when the API is frozen.
Comment 1 Michael Catanzaro 2022-12-22 11:34:30 PST
Pull request: https://github.com/WebKit/WebKit/pull/8011
Comment 2 Michael Catanzaro 2023-01-05 13:58:24 PST
Closing since Carlos Lopez didn't like this idea.