Skip to content

[CMake] Fix ANGLE for Mac CMake build#62560

Merged
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
pascoej:eng/cmake-mac-angle
Apr 13, 2026
Merged

[CMake] Fix ANGLE for Mac CMake build#62560
webkit-commit-queue merged 1 commit intoWebKit:mainfrom
pascoej:eng/cmake-mac-angle

Conversation

@pascoej
Copy link
Copy Markdown
Member

@pascoej pascoej commented Apr 11, 2026

9b5e74e

[CMake] Fix ANGLE for Mac CMake build
https://bugs.webkit.org/show_bug.cgi?id=312022
rdar://problem/174562352

Reviewed by BJ Burg.

Set is_apple flag, update MSL source variable name, and add NO_SYMLINK
to header copy targets for CMake Mac builds.

Based on a base patch by Simon Lewis.

* Source/ThirdParty/ANGLE/CMakeLists.txt: Add is_apple flag inside
if (APPLE) block so ANGLE's GN-derived cmake finds Apple sources.
* Source/ThirdParty/ANGLE/PlatformMac.cmake: Rename
angle_translator_lib_metal_sources to angle_translator_lib_msl_sources
to match upstream ANGLE variable rename.
* Source/ThirdParty/ANGLE/include/CMakeLists.txt: Add NO_SYMLINK to
ANGLEWebGLHeaders and GLSLANGHeaders so adjust-angle-include-paths.py
does not rewrite source tree files.

Canonical link: https://commits.webkit.org/311107@main

387d128

Misc iOS, visionOS, tvOS & watchOS macOS Linux Windows Apple Internal
✅ 🧪 style ✅ 🛠 ios ✅ 🛠 mac ✅ 🛠 wpe 🛠 win ⏳ 🛠 ios-apple
✅ 🛠 ios-sim ✅ 🛠 mac-AS-debug ✅ 🧪 wpe-wk2 🧪 win-tests ⏳ 🛠 mac-apple
✅ 🧪 webkitperl ✅ 🧪 ios-wk2 ✅ 🧪 api-mac ✅ 🧪 api-wpe ⏳ 🛠 vision-apple
🧪 ios-wk2-wpt 🧪 api-mac-debug ✅ 🛠 gtk3-libwebrtc
✅ 🧪 api-ios ✅ 🧪 mac-wk1 ✅ 🛠 gtk
✅ 🧪 mac-wk2 ✅ 🧪 gtk-wk2
✅ 🛠 vision ✅ 🧪 mac-AS-debug-wk2 ✅ 🧪 api-gtk
✅ 🛠 🧪 merge ✅ 🛠 vision-sim ✅ 🧪 mac-wk2-stress ✅ 🛠 playstation
✅ 🧪 vision-wk2 ✅ 🧪 mac-intel-wk2
✅ 🛠 tv
✅ 🛠 tv-sim
✅ 🛠 watch
✅ 🛠 watch-sim

@pascoej pascoej self-assigned this Apr 11, 2026
@pascoej pascoej added the CMake Bugzilla component for CMake build system changes label Apr 11, 2026
@webkit-ews-buildbot webkit-ews-buildbot added the merging-blocked Applied to prevent a change from being merged label Apr 12, 2026
@pascoej pascoej removed the merging-blocked Applied to prevent a change from being merged label Apr 12, 2026
@pascoej pascoej marked this pull request as ready for review April 12, 2026 05:31
@pascoej pascoej force-pushed the eng/cmake-mac-angle branch from 51cd91b to 6473508 Compare April 12, 2026 19:49
@@ -44,15 +44,15 @@ if (ENABLE_WEBGL)
WEBKIT_COPY_FILES(ANGLEWebGLHeaders
DESTINATION ${ANGLE_FRAMEWORK_HEADERS_DIR}/ANGLE
FILES ${ANGLE_WEBGL_HEADERS}
FLATTENED
FLATTENED NO_SYMLINK # adjust-angle-include-paths.py rewrites these in-place
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice comment!

@pascoej pascoej force-pushed the eng/cmake-mac-angle branch from 6473508 to 387d128 Compare April 13, 2026 15:07
@pascoej pascoej added the merge-queue Applied to send a pull request to merge-queue label Apr 13, 2026
https://bugs.webkit.org/show_bug.cgi?id=312022
rdar://problem/174562352

Reviewed by BJ Burg.

Set is_apple flag, update MSL source variable name, and add NO_SYMLINK
to header copy targets for CMake Mac builds.

Based on a base patch by Simon Lewis.

* Source/ThirdParty/ANGLE/CMakeLists.txt: Add is_apple flag inside
if (APPLE) block so ANGLE's GN-derived cmake finds Apple sources.
* Source/ThirdParty/ANGLE/PlatformMac.cmake: Rename
angle_translator_lib_metal_sources to angle_translator_lib_msl_sources
to match upstream ANGLE variable rename.
* Source/ThirdParty/ANGLE/include/CMakeLists.txt: Add NO_SYMLINK to
ANGLEWebGLHeaders and GLSLANGHeaders so adjust-angle-include-paths.py
does not rewrite source tree files.

Canonical link: https://commits.webkit.org/311107@main
@webkit-commit-queue
Copy link
Copy Markdown
Collaborator

Committed 311107@main (9b5e74e): https://commits.webkit.org/311107@main

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

@webkit-commit-queue webkit-commit-queue merged commit 9b5e74e into WebKit:main Apr 13, 2026
@webkit-commit-queue webkit-commit-queue removed the merge-queue Applied to send a pull request to merge-queue label Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMake Bugzilla component for CMake build system changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants