Skip to content

Conversation

@dougiesquire
Copy link
Collaborator

@dougiesquire dougiesquire commented May 13, 2025

The changes in this PR allow libaccessom2 to be used in other CMake projects using find_package:

  • CMake targets are exported
  • CMake package config file is configured and installed
  • CMAKE_INSTALL_INCLUDEDIR is added to accessom2 include directories

Copy link
Member

@micaeljtoliveira micaeljtoliveira left a comment

Choose a reason for hiding this comment

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

I can't see any obvious issues in the added code, so will approve.

@dougiesquire
Copy link
Collaborator Author

Thanks @micaeljtoliveira. I'll wait for @harshula's nod of approval before merging.

@harshula
Copy link

harshula commented May 15, 2025

I'm not familiar enough with the CMake side to review this, so please go ahead and merge since @micaeljtoliveira has reviewed it.

@dougiesquire dougiesquire merged commit d53200e into master May 15, 2025
4 checks passed
@dougiesquire dougiesquire deleted the cmake-export branch May 15, 2025 00:46
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.

4 participants