-
Notifications
You must be signed in to change notification settings - Fork 3
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
Move playground.cpp to a sandbox module #48
Comments
there's really just one test that needs In principle, the integration tests shouldn't depend on "egrabber.h", imo. |
I'm generally in favor of putting things like that in a sandbox module that you have to turn on behind a CMake flag. Happy to do this if that's the way you want to take it. |
-DNOTEST=1
We can't build tests on unsupported platforms because they require
EGrabber.h
. We can add theNOTEST
flag when we configure in order to get around this, but we could also just setNOTEST=1
when we can't find the EGrabber library.The text was updated successfully, but these errors were encountered: