Skip to content

Include xyz_value_types-export-set.cmake in package config file#625

Merged
Twon merged 1 commit into
jbcoe:mainfrom
jogerh:fix-install
Feb 24, 2026
Merged

Include xyz_value_types-export-set.cmake in package config file#625
Twon merged 1 commit into
jbcoe:mainfrom
jogerh:fix-install

Conversation

@jogerh

@jogerh jogerh commented Feb 23, 2026

Copy link
Copy Markdown
Contributor

CMake fails to find 'xyz_value_types-target.cmake' because it does not exist. It seems like xyz_value_types-export-set.cmake should be included instead.

Fixes #624

CMake fails to find 'xyz_value_types-target.cmake' because it does not exist. It seems like `xyz_value_types-export-set.cmake` should be included instead.

Fixes jbcoe#624
@jogerh jogerh requested review from Twon and jbcoe as code owners February 23, 2026 21:02
@jbcoe

jbcoe commented Feb 23, 2026

Copy link
Copy Markdown
Owner

@Twon - can you take a look? Looks good to me.

@Twon Twon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks right to me. Thanks for the contribution!

@Twon Twon merged commit 6f50aff into jbcoe:main Feb 24, 2026
2 checks passed
@jogerh

jogerh commented Feb 24, 2026

Copy link
Copy Markdown
Contributor Author

Great! Thanks @Twon.

@jbcoe and @Twon, Do you have any plans to create a new release of value_types anytime soon, so we could get a new vcpkg with this fix?

@jbcoe

jbcoe commented Feb 24, 2026

Copy link
Copy Markdown
Owner

That sounds like a great idea.

@Twon

Twon commented Feb 24, 2026

Copy link
Copy Markdown
Collaborator

I've created a new release here: https://github.com/jbcoe/value_types/releases/tag/v1.0.1

@jogerh

jogerh commented Feb 24, 2026

Copy link
Copy Markdown
Contributor Author

I created a PR to update vcpkg. Does it look OK to you? microsoft/vcpkg#50156

@jogerh jogerh deleted the fix-install branch February 24, 2026 23:07
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.

find_package(xyz_value_types CONFIG REQUIRED) fails with vcpkg and locally installed build

3 participants