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

Add cmake interface support #2033

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

caseymcc
Copy link

  • When adding a feature, please describe the usage context (how you intend to use it, why you need it, etc.).

Added support for cmake install using library interface. Added to support Hunter package manager, https://github.com/ruslo/hunter. Specifics for imgui are at, ruslo/hunter#1521. Once Hunter's PR is accepted and a release created there will be package documentation at https://docs.hunter.sh/en/latest/packages/pkg/imgui.html.

This PR is not required for Hunter to work as required changes are hosted here https://github.com/hunter-packages/imgui, however it is better when the main line supports it and the changes are not Hunter specific just cmake. Also I can create a PR for a Hunter Badge for imgui's readme if this is something you all want.

@ocornut
Copy link
Owner

ocornut commented Aug 20, 2018

Hello @caseymcc and thanks for the PR. See #1713

@caseymcc
Copy link
Author

Sounds good, ill peek at it once it is accepted and make changes to it if needed.

@thedmd thedmd mentioned this pull request Dec 5, 2019
@ocornut ocornut force-pushed the master branch 2 times, most recently from 0c1e5bd to bb6a60b Compare August 27, 2021 19:10
@ocornut ocornut force-pushed the master branch 2 times, most recently from 8b83e0a to d735066 Compare December 13, 2021 11:31
@ocornut ocornut force-pushed the master branch 2 times, most recently from b3b85d8 to 0755767 Compare January 17, 2022 14:21
@ocornut ocornut force-pushed the master branch 3 times, most recently from c817acb to 8d39063 Compare February 15, 2022 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants