Skip to content

Deprecate environment variable WEBKIT_BUILD_USE_SYSTEM_LIBRARIES.#222

Open
clopez wants to merge 1 commit intoIgalia:mainfrom
clopez:delete_WEBKIT_BUILD_USE_SYSTEM_LIBRARIES
Open

Deprecate environment variable WEBKIT_BUILD_USE_SYSTEM_LIBRARIES.#222
clopez wants to merge 1 commit intoIgalia:mainfrom
clopez:delete_WEBKIT_BUILD_USE_SYSTEM_LIBRARIES

Conversation

@clopez
Copy link
Copy Markdown
Collaborator

@clopez clopez commented Mar 19, 2026

This is not longer needed since https://commits.webkit.org/309557@main
The env var used by the WebKit tooling to identify if is running inside the SDK is now WEBKIT_CONTAINER_SDK.

However, to allow building older versions of WebKit with the SDK, instead of just removing the variable now, add a comment about the deprecation status with the idea of removing it on the future.

@clopez clopez requested a review from philn March 19, 2026 15:19
@TingPing
Copy link
Copy Markdown
Member

You can't immediately remove this because of a change from 10 hours ago. People will be building the past few years of commits inside the sdk.

@clopez
Copy link
Copy Markdown
Collaborator Author

clopez commented Mar 20, 2026

You can't immediately remove this because of a change from 10 hours ago. People will be building the past few years of commits inside the sdk.

mmmm, good point.

then let's keep it but add a note about it being deprecated

@clopez
Copy link
Copy Markdown
Collaborator Author

clopez commented Mar 20, 2026

I have also marked WebKit/WebKit#60862 as suggested to be backported to 2.50 and 2.52

This is not longer needed since https://commits.webkit.org/309557@main
The env var used by the WebKit tooling to identify if is running inside
the SDK is now WEBKIT_CONTAINER_SDK.

However, to allow building older versions of WebKit with the SDK,
instead of just removing the variable now, add a comment about the
deprecation status with the idea of removing it on the future.
@clopez clopez force-pushed the delete_WEBKIT_BUILD_USE_SYSTEM_LIBRARIES branch from 6c862d4 to 5bff5d2 Compare March 20, 2026 10:28
@clopez clopez changed the title Stop setting environment variable WEBKIT_BUILD_USE_SYSTEM_LIBRARIES. Deprecate environment variable WEBKIT_BUILD_USE_SYSTEM_LIBRARIES. Mar 20, 2026
@clopez clopez requested a review from TingPing March 20, 2026 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants