Skip to content

[robotraconteur-companion] Add patch to fix eigen3 5 version in cmake package config files#51231

Merged
BillyONeal merged 2 commits into
microsoft:masterfrom
johnwason:robotraconteur_companion_eigen_cmake_pkg_patch
Apr 20, 2026
Merged

[robotraconteur-companion] Add patch to fix eigen3 5 version in cmake package config files#51231
BillyONeal merged 2 commits into
microsoft:masterfrom
johnwason:robotraconteur_companion_eigen_cmake_pkg_patch

Conversation

@johnwason

Copy link
Copy Markdown
Contributor
  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version, or no changes were necessary.
  • Any fixed CI baseline and CI feature baseline entries are removed from that file, or no entries needed to be changed.
  • All patch files in the port are applied and succeed.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Exactly one version is added in each modified versions file.

This PR adds a patch to the robotraconteur-companion package to fix finding eigen3 version 5 or higher in the cmake package configuration files.

Comment on lines +9 to +13
-find_package(Eigen3 3.3 REQUIRED NO_MODULE)
+find_package(Eigen3 3.3 NO_MODULE QUIET)
+if(NOT Eigen3_FOUND)
+ find_package(Eigen3 3.3...<6 REQUIRED NO_MODULE)
+endif()

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.

Integrate the change, and align the approach, with 0001-support-eigen3-5.patch. It is one topic.

@johnwason johnwason force-pushed the robotraconteur_companion_eigen_cmake_pkg_patch branch from 3c59768 to 0eb192f Compare April 18, 2026 03:18
@johnwason

Copy link
Copy Markdown
Contributor Author

I updated the patch as requested

@BillyONeal BillyONeal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@BillyONeal BillyONeal merged commit ed7bb6e into microsoft:master Apr 20, 2026
16 checks passed
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.

3 participants