Don't run postbuild when we detect electron install

maxkorp · maxkorp · commit 14f4a7aa3972 · 2016-09-14T19:41:46.000-07:00
Since we can't require it from node if it's compiled for electron, we should just assume success for now
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) {
+    // If we're building for electron, we're unable to require things so we should
+    // 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,8 @@ 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,
+  mustBuild: !!(isGitRepo || process.env.BUILD_DEBUG || process.env.BUILD_ONLY)
 };
