-
Notifications
You must be signed in to change notification settings - Fork 60
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
track slick #12
Comments
fixed by #137 for the 2.12 build |
@szeiger on 2.11 I get the following. any idea what this is about?
this is from https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-community-build/85/console looks like the same thing when building on JDK8 too: https://scala-ci.typesafe.com/job/scala-2.11.x-jdk8-integrate-community-build/54/consoleFull |
@szeiger ping |
Slick 3.2 will require Java 8, so I upgraded the build to Java 8 and also upgraded the OSGi test environment because the old version was broken on Java 8. That said, I can still build it and successfully run the OSGi tests locally under Java 7. The osgitests project is the only one with The error on JDK8 seems to be a different one:
I cannot reproduce the actual test failure locally, either, but I can reproduce the duplicate entry for reactive-streams:
I'll try to fix this and submit a PR to Slick. (There's yet another OSGi test related error in the RP dbuild which we couldn't figure out, so we ended up disabling the OSGi tests there.) |
This causes a test failure in the Scala community build (scala/community-build#12) even though it does not affect the OSGi tests in the Slick project directly.
thanks, this is very helpful. ok, it sounds like the OSGi tests are somewhat fragile, at least in a dbuild context, and I'm in shortcut-taking mode on community build stuff so I can devote more attention to other things, so I think I'll just disable the OSGi tests on any branches where they're failing. since master now expects Java 8, I'll change the 2.11.x/jdk6 to use the 3.1 branch instead of master. |
The problem on JDK8 with Slick master should now be fixed/ |
progress: #138 |
currently at "slick/slick.git#pull/686/head"
The text was updated successfully, but these errors were encountered: