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

[highfive] Add eigen3 feature option #21897

Merged
merged 13 commits into from
Dec 10, 2021

Conversation

JonLiu1993
Copy link
Member

@JonLiu1993 JonLiu1993 commented Dec 7, 2021

Describe the pull request

Add Eigen3 feature option such that highfive[eigen3] is available.

Execpt feature "test", all features are tested successfully in the following triplet:

@JonLiu1993 JonLiu1993 added category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:internal This PR or Issue was filed by the vcpkg team. labels Dec 7, 2021
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 431d3810aa9241aed49b14367be417f225d2af5e -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/h-/highfive.json b/versions/h-/highfive.json
index bebffde..3d98b98 100644
--- a/versions/h-/highfive.json
+++ b/versions/h-/highfive.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "ccdcfa177fc48552d13cee2a9759d4fb14dbbe3a",
+      "git-tree": "c014dff160e047536190f8ae97b12b33550eda16",
       "version": "2.3",
       "port-version": 1
     },

@JonLiu1993 JonLiu1993 marked this pull request as ready for review December 8, 2021 02:27
ports/highfive/portfile.cmake Outdated Show resolved Hide resolved
ports/highfive/portfile.cmake Outdated Show resolved Hide resolved
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 431d3810aa9241aed49b14367be417f225d2af5e -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/h-/highfive.json b/versions/h-/highfive.json
index 52aaf28..625d311 100644
--- a/versions/h-/highfive.json
+++ b/versions/h-/highfive.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "3841a549b9308221ed9a5201dadbb6ec1955907e",
+      "git-tree": "d6233178af08a74ff1637956504e2039f51e6a8a",
       "version": "2.3",
       "port-version": 1
     },

@JackBoosY JackBoosY added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Dec 9, 2021
@BillyONeal BillyONeal added requires:author-response and removed info:reviewed Pull Request changes follow basic guidelines labels Dec 10, 2021
@JackBoosY JackBoosY added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Dec 10, 2021
@BillyONeal BillyONeal merged commit 5420dab into microsoft:master Dec 10, 2021
@BillyONeal
Copy link
Member

Thanks for your help!

@JonLiu1993 JonLiu1993 deleted the dev/Jon/highfive branch December 13, 2021 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist info:internal This PR or Issue was filed by the vcpkg team. info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[highfive] Eigen3 feature option
3 participants