Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.x] Upgrade Embree to the latest official release. #48967

Merged
merged 1 commit into from
May 24, 2021

Conversation

akien-mga
Copy link
Member

Since Embree v3.13.0 supports AARCH64, switch back to the
official repo instead of using Embree-aarch64.

thirdparty/embree/patches/godot-changes.patch should now contain
an accurate diff of the changes done to the library.

Backport of #48885, including a fix for #48822.

Since Embree v3.13.0 supports AARCH64, switch back to the
official repo instead of using Embree-aarch64.

`thirdparty/embree/patches/godot-changes.patch` should now contain
an accurate diff of the changes done to the library.

(cherry picked from commit 767e374)
@akien-mga akien-mga added bug enhancement topic:thirdparty cherrypick:3.x Considered for cherry-picking into a future 3.x release labels May 22, 2021
@akien-mga akien-mga added this to the 4.0 milestone May 22, 2021
@akien-mga akien-mga requested review from a team as code owners May 22, 2021 13:17
@akien-mga akien-mga changed the title Upgrade Embree to the latest official release. [3.x] Upgrade Embree to the latest official release. May 22, 2021
@akien-mga
Copy link
Member Author

I made a test build (classical only, and no UWP) for this PR to help validate the upgrade (and especially make sure that it works fine with MinGW builds on Windows):

https://downloads.tuxfamily.org/godotengine/testing/3.4-beta-embree3.13.0/

I'd love to know from lightmap users that:

  • It works fine on Windows
  • It works fine on macOS ARM64
  • It works as well (or better!) than the version in 3.3-stable and 3.3.1-stable

Note: It's a build based on the 3.x branch, i.e. 3.4-beta. At this stage I don't know of any change that would break compatibility with 3.3, but if you want to be on the safe side I would advise to use VCS or do a backup before opening a 3.3.x project with this build (so you can go back to 3.3.x after testing).

@akien-mga akien-mga added cherrypick:3.3 and removed cherrypick:3.x Considered for cherry-picking into a future 3.x release labels May 23, 2021
@akien-mga akien-mga merged commit 65ddfd9 into godotengine:3.x May 24, 2021
@akien-mga akien-mga deleted the 3.x-embree-3.13.0 branch May 24, 2021 15:03
@YuriSizov YuriSizov modified the milestones: 4.0, 3.x Feb 4, 2023
@YuriSizov YuriSizov modified the milestones: 3.x, 3.4 Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants