Changing purejavacomm source to avoid insecure http repo #10
Conversation
|
I'm not sure about that travis failure... says it can't find "lein2." |
|
Hmm, well looks like Java 1.7.0_151 OpenJDK doesn't play nice with the github version of purejavacomm. This PR probably isn't going to work out so I'll close it and leave the issue to you. |
|
Trying 1.0.2 |
|
Still not working with openjdk7... |
|
It looks like a version mismatch. purejavacomm is apparently compiled with Java 8 (which maps to major version 52 according to http://javarevisited.blogspot.com/2015/05/fixing-unsupported-majorminor-version.html). Weird that it works in openjdk6 and oraclejdk7... |
WhittlesJr
force-pushed
the
whittlesjr-purejavacommrepo
branch
from
b13427d
to
2814900
Compare
|
This (the Java 8 issue) is something I struggled with in #7 as well. Changing to 1.0.1 is actually how we worked around it at the time, as 1.0.0 was released for Java 8 only. Regarding openjdk6 comment if you look at the travis run it didn't swap to jdk6 it was still running 1.8.0_151 |
|
Is there a way to use lein profiles for the various jdks? |
|
@peterschwarz I don't know of one, it appears that OpenJDK 6 was removed from the trusty image and needs to be specifically installed now. travis-ci/travis-ci#8199 (comment) |
|
Shame that the developer didn't use maven classifiers for that. Will do some research. |
|
Could probably update the travis.yaml with the correct set of JDK options in a separate PR |
|
I just sent one in to try that for JDK6, I should have just enabled travis on my clone rather then PR'ing it first... (Sorry) |
|
Going to merge in #11 and then you can see if that fixes this issue. If so, I will close this PR and the corresponding issue. |
|
The issue seems to be resolve, so I will close this PR |
Potential fix for issue #9