Don't run postbuild when we detect electron install

maxkorp · Maximiliano Korp · commit b18ef4cc44a6 · 2016-10-03T09:53:34.000-07:00
diff --git a/lifecycleScripts/postinstall.js b/lifecycleScripts/postinstall.js
@@ -24,6 +24,11 @@ module.exports = function install() {
     // cleaning up
     return Promise.resolve();
   }
+  if (buildFlags.isElectron || buildFlags.isNWjs) {
+    // If we're building for electron or NWjs, we're unable to require the
+    // built library so we have to just assume success, unfortunately.
+    return Promise.resolve();
+  }
 
   return exec("node " + path.join(rootPath, "dist/nodegit.js"))
     .catch(function(e) {
diff --git a/utils/buildFlags.js b/utils/buildFlags.js
@@ -11,7 +11,9 @@ try {
 }
 
 module.exports = {
-  debugBuild: process.env.BUILD_DEBUG,
+  debugBuild: !!process.env.BUILD_DEBUG,
+  isElectron: process.env.npm_config_runtime === "electron",
   isGitRepo: isGitRepo,
-  mustBuild: isGitRepo || process.env.BUILD_DEBUG || process.env.BUILD_ONLY,
+  isNwjs: process.env.npm_config_runtime === "node-webkit",
+  mustBuild: !!(isGitRepo || process.env.BUILD_DEBUG || process.env.BUILD_ONLY)
 };
