Merge pull request #310 from justadudewhohacks/fixes

bump nan version to fix node v10 issues + fix windows error message i…

Author: justadudewhohacks

lib/commons.js

@@ -11,7 +11,11 @@ function resolvePath(filePath, file) {
 const defaultDir = '/usr/local';
 
 function getLibDir() {
-  return resolvePath(process.env.OPENCV_LIB_DIR) || `${defaultDir}/lib`;
+  const libPath = resolvePath(process.env.OPENCV_LIB_DIR)
+  if (process.platform === 'win32' && !libPath) {
+    throw new Error('OPENCV_LIB_DIR is not defined')
+  }
+  return libPath || `${defaultDir}/lib`;
 }
 
 module.exports = {

package-lock.json

@@ -39,7 +39,7 @@
   "gypfile": true,
   "dependencies": {
     "macro-inferno": "^0.2.1",
-    "nan": "^2.7.0",
+    "nan": "^2.10.0",
     "native-node-utils": "0.1.2",
     "opencv-build": "^0.0.12"
   },