Adding a mechanism to add the jvm binary to PATH.

jsdevel · jsdevel · commit cab373f6bb7b · 2014-05-14T08:55:27.000-07:00
diff --git a/README.md b/README.md
@@ -28,13 +28,9 @@ $ npm install java
 ```
 
 Notes:
-* If you see an error such as "Error: The specified module could not be found. 
-      xxxxxx\node_modules\java\build\Release\nodejavabridge_bindings.node".
-      Ensure the directory where jvm.dll exists is in your PATH. (e.g. C:\Program Files (x86)\Java\jdk1.6.0_18\jre\bin\client).
-      This path cannot have quotes.
 * node-gyp requires python 2.x not python 3.x. See https://github.com/TooTallNate/node-gyp/issues/155 for more details.
 * For 64 bit installs with 32 bit node
- * you need the 32 bit JDK, with the 64 bit JDK you will see LNK2001   
+ * you need the 32 bit JDK, with the 64 bit JDK you will see LNK2001
    errormessages (http://stackoverflow.com/questions/10309304/what-library-to-link-to-on-windows-7-for-jni-createjavavm).
  * when using the windows SDK 7.1 command prompt (64 bits) be sure to setenv.cmd /Release /x86
 * If you see an error such as "Call to 'node findJavaHome.js' returned exit status 1"
diff --git a/lib/nodeJavaBridge.js b/lib/nodeJavaBridge.js
@@ -1,5 +1,7 @@
 'use strict';
 
+process.env.PATH += require('../build/jvm_dll_path.json');
+
 var path = require('path');
 var binaryPath = path.resolve(path.join(__dirname, "../build/Release/nodejavabridge_bindings.node"));
 var bindings = require(binaryPath);
diff --git a/package.json b/package.json
@@ -26,15 +26,17 @@
     "url": "https://github.com/joeferner/node-java.git"
   },
   "dependencies": {
-    "find-java-home": "0.0.7"
+    "find-java-home": "0.0.7",
+    "glob": "~3.2.9"
   },
   "devDependencies": {
     "nodeunit": "~0.6.4",
     "memwatch": "~0.2.0",
     "async": "~0.1.22"
   },
   "scripts": {
-    "test": "nodeunit test"
+    "test": "nodeunit test",
+    "postinstall": "node postInstall.js"
   },
   "main": "./index.js"
 }
diff --git a/postInstall.js b/postInstall.js
@@ -0,0 +1,25 @@
+var glob = require('glob');
+var fs = require('fs');
+var path = require('path');
+
+require('find-java-home')(function(err, home){
+  var dll;
+  var so;
+  var binary;
+
+  if(home){
+    dll = glob.sync('**/jvm.dll', {cwd: home})[0];
+    so = glob.sync('**/libjvm.so', {cwd: home})[0];
+    binary = dll || so;
+
+    fs.writeFileSync(
+      path.resolve(__dirname, './build/jvm_dll_path.json'),
+      binary
+      ? JSON.stringify(
+          path.delimiter
+          + path.dirname(path.resolve(home, binary))
+        )
+      : '""'
+    );
+  }
+});
