Skip to content

Updated bgfx.cmake to latest.#1669

Merged
bkaradzic-microsoft merged 1 commit intoBabylonJS:masterfrom
bkaradzic-microsoft:update-bgfx-cmake
Apr 21, 2026
Merged

Updated bgfx.cmake to latest.#1669
bkaradzic-microsoft merged 1 commit intoBabylonJS:masterfrom
bkaradzic-microsoft:update-bgfx-cmake

Conversation

@bkaradzic-microsoft
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the pinned bgfx.cmake FetchContent dependency revision used by Babylon Native’s CMake build to a newer upstream commit.

Changes:

  • Bump bgfx.cmake GIT_TAG from a0adee5... to be578c80....

@bkaradzic-microsoft bkaradzic-microsoft merged commit 063ea09 into BabylonJS:master Apr 21, 2026
32 checks passed
bghgary added a commit to bghgary/BabylonNative that referenced this pull request Apr 22, 2026
…erResourceView

The createTexture2D _external path now works on WARP after the bgfx update
in master. Drop the placeholder + AfterRenderScheduler + overrideInternal
two-step dance and the extra frame pump that existed to apply it. This also
restores full array-slice SRV so RenderTextureArray no longer needs a
sanitizer-only skip.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
bghgary added a commit to bghgary/BabylonNative that referenced this pull request Apr 22, 2026
The SKIP_RENDER_TESTS flag was originally added because WARP could not handle the _external path in bgfx. bgfx PR BabylonJS#1669 fixed CreateShaderResourceView for the _external parameter, so the render test should pass on WARP now. Let CI verify.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bkaradzic-microsoft bkaradzic-microsoft deleted the update-bgfx-cmake branch April 23, 2026 00:24
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.

4 participants