vert-x3 / vertx-lang-js

Nashorn JavaScript implementation for Vert.x
Apache License 2.0
35 stars 23 forks source link

Add check for Java.synchronized #27

Closed purplefox closed 9 years ago

purplefox commented 9 years ago

Java.synchronized is used in jvm-npm.js to sychronize concurrent access to require but this is only available in JDK 8u45 or later.

We should explicitly check for Java.synchronized is undefined and log a helpful warning e.g. "please upgrade to 8u45 or later" rather than the cryptic error that now occurs.

We should also mention in the documentation that 8u45 or later is required for JS.