Skip to content
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

sshj can't run with java 6 #234

Closed
sblond opened this issue Feb 10, 2016 · 6 comments
Closed

sshj can't run with java 6 #234

sblond opened this issue Feb 10, 2016 · 6 comments

Comments

@sblond
Copy link

sblond commented Feb 10, 2016

Hi,

I get java.lang.UnsupportedClassVersionError: net/i2p/crypto/eddsa/spec/EdDSAParameterSpec : Unsupported major.minor version 51.0 when using sshj 0.15.0 with java 6. Usually this error means that a class has been compiled with java 7 compatibility but run with an older java.

This may be related to the fix of #171 that adds a new dependency to net.vrallev.ecc:ecc-25519-java:1.0.1. This dependency seems to require the use of java 7 https://github.com/vRallev/ECC-25519/blob/master/ECC-25519-Java/build.gradle.

Does it mean that java 6 compatibility is dropped?

@hierynomus
Copy link
Owner

Hi Stephane,

Thanks for pointing that out. Hadn't actually noticed that!
I am somewhat inclined to then maybe indeed drop java6 support for new versions. Or to make ecc-25519 an optional dependency, and checking in the code whether it can be added or not.
What would be the preference?

@dkocher
Copy link
Contributor

dkocher commented Feb 24, 2016

+1 to drop Java 6 support.

@sblond
Copy link
Author

sblond commented Feb 25, 2016

I have to deal with java 6, so
+1 to keep java 6 support

@hierynomus
Copy link
Owner

I cannot easily make ecc-25519 optional, as it is used directly in the KeyType enum, so I cannot optionally load it. So I am dropping java6 support, sorry.

@hierynomus
Copy link
Owner

@sblond We've been able to add Java6 support back to the mix.

@sblond
Copy link
Author

sblond commented Nov 9, 2016

Hi,
I upgraded to 0.18.0 and it worked fine.
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants