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

[embree3] Update to 3.11.0 #12555

Merged
merged 4 commits into from
Aug 9, 2020
Merged

Conversation

regen100
Copy link
Contributor

Update to the latest release v3.11.0.
Outdated patches are replaced.

@ghost
Copy link

ghost commented Jul 24, 2020

CLA assistant check
All CLA requirements met.

@NancyLi1013 NancyLi1013 added the category:port-update The issue is with a library, which is requesting update new revision label Jul 24, 2020
@regen100 regen100 marked this pull request as ready for review July 24, 2020 09:48
Remove unused function include(vcpkg_common_functions)
@NancyLi1013
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

ports/embree3/portfile.cmake Show resolved Hide resolved
ports/embree3/portfile.cmake Outdated Show resolved Hide resolved
@NancyLi1013
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@NancyLi1013
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@NancyLi1013 NancyLi1013 added the info:reviewed Pull Request changes follow basic guidelines label Aug 5, 2020
@@ -0,0 +1,4 @@
The package embree3 provides CMake targets:

find_package(embree 3 CONFIG REQUIRED)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
find_package(embree 3 CONFIG REQUIRED)
find_package(embree3 CONFIG REQUIRED)

Isn't this the correct find_package?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. Related to #8608, users often use find_package(embree 3).
And also, Upstream README says using FIND_PACKAGE(embree 3.0 REQUIRED).

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, alright, cool :)

@strega-nil strega-nil merged commit c7a2f02 into microsoft:master Aug 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-update The issue is with a library, which is requesting update new revision info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants