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 option to enable/disable example generation #37

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

Conversation

BenjaminBeichler
Copy link

fixes part of #23

Signed-off-by: Benjamin Beichler [email protected]

@BenjaminBeichler
Copy link
Author

I have just done the trivial rebase for my old pull request. Are there any other obstacles ?

@maehne was in favor for this in #23, so maybe you can push it ;-)

@markfoodyburton
Copy link
Contributor

As an interim solution, I'd go with this. (It would help).
Ideally we should move to a more 'cmake/ctest' way of doing things.

@BenjaminBeichler
Copy link
Author

BenjaminBeichler commented Feb 8, 2024

Yes, agree. But all the cmake files would need updates. I opted for the minimal invasive version.

@markfoodyburton
Copy link
Contributor

(Actually there's work going on to fix it properly - should be merged soon :-) )

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.

2 participants