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.

Sign up for GitHub

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

Jump to bottom

CMake: cannot install target doctest_with_main #310

Closed
theodelrieu opened this issue Nov 18, 2019 · 2 comments
Closed

CMake: cannot install target doctest_with_main #310

theodelrieu opened this issue Nov 18, 2019 · 2 comments

Comments

@theodelrieu
Copy link

theodelrieu commented Nov 18, 2019 •
edited by onqtam
Loading

Hello,

I'm currently trying to package doctest for conan-center-index, but I'm struggling with the target doctest_with_main, which has the property EXCLUDE_FROM_ALL.

This property prevents the target from being installed normally. Is this a wanted behavior?
@onqtam
Copy link
Member

onqtam commented Nov 21, 2019

I don't think it's necessary to package doctest_with_main - the user could easily make some .cpp file and just define DOCTEST_CONFIG_IMPLEMENT_WITH_MAIN before including doctest.h.

The EXCLUDE_FROM_ALL property is there so it doesn't get compiled by default when the CMakeLists.txt is used as a subproject - not everyone would need it.

@theodelrieu theodelrieu mentioned this issue Nov 21, 2019
Merged
4 tasks
@onqtam
Copy link
Member

onqtam commented Dec 16, 2019

I'm closing this since I don't think anything more needs to be done here :)

@onqtam onqtam closed this as completed Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@onqtam @theodelrieu