-
Notifications
You must be signed in to change notification settings - Fork 33
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
The jpl:prolog_in_java test stopped working for 9.2.7 #103
Comments
I'm not much of a Java expert, but I suspect that the "hamcrest" Java library is missing. It probably just skipped the test before and, as the rules for detecting junit have been extended it now tries to execute this test. You need junit 4 and hamcrest for this test to work. |
@JanWielemaker Thanks for the tip! Hamcrest seems to be an abandoned project, though? This package is orphaned: https://archlinux.org/packages/extra/any/java-hamcrest/ Probably because the upstream project is archived: https://code.google.com/archive/p/hamcrest/source Do you happen to know if there are any plans to move away from hamcrest in connection with the tests? |
(Un)fortunately I know very little about Java. Including @anionic. Paul, do you know how to resolve this? I do not really understand this though. If I read packages/jpl/CMakeLists.txt and packages/jpl/cmake/JUnit.cmake correctly it tries to find HAMCREST and if that fails it skips the prolog_in_java test. Do you have the log of the cmake run to see what it says about junit and hamcrest? |
Depending on junit and Hamcrest when testing solved the issue. Thanks! I guess a feature request for not depending on the discontinued Hamcrest project belongs in a separate issue. |
I celebrated too early. It did not work after all, because I forgot to remove the |
Thanks for trying. The issue is still open, no? |
Yes, I believe so. |
Hi, maintainer of the
swi-prolog
Arch Linux package here.When upgrading the package from 9.2.4 to 9.2.7, one of the tests stopped working:
Here is the current PKGBUILD:
Any advice for how to make the tests pass is welcome.
The text was updated successfully, but these errors were encountered: