-
-
Notifications
You must be signed in to change notification settings - Fork 36
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
Using compiler in test phase #41
Comments
I tried to add the compiler tag only to the test section but the .ci_support files did not get generated correctly. Instead of creating entries for linux and osx it only generated them for windows. |
Here is the commit of the regenerated support files. |
Perfect. Was just about to ask for that. Can you please raise an issue on the |
I was expecting the gcc7 and toolchain variants for linux. Maybe that will get picked up by the bot later.. |
Ah ok. Yeah, that's correct. The bot will take care of that afterwards. |
As the test phase of the recipe is making use of a C compiler for JNI testing, it would be good to use the compiler syntax to require one and update the test to be a bit more general about the compiler used. Should help make sure that JNI works nicely with the compilers we use to build other packages (including those that will make use of JNI).
The text was updated successfully, but these errors were encountered: