Browse Source

Inline platform variable

Kevin Sawicki 8 years ago
parent
commit
ed8260e4d9
1 changed files with 21 additions and 21 deletions
  1. 21 21
      npm/install.js

+ 21 - 21
npm/install.js

@@ -16,27 +16,7 @@ try {
   // do nothing
 }
 
-var platform = process.env.npm_config_platform || os.platform()
-
-function onerror (err) {
-  throw err
-}
-
-function getPath (platform) {
-  switch (platform) {
-    case 'darwin':
-      return 'dist/Electron.app/Contents/MacOS/Electron'
-    case 'freebsd':
-    case 'linux':
-      return 'dist/electron'
-    case 'win32':
-      return 'dist/electron.exe'
-    default:
-      throw new Error('Electron builds are not available on platform: ' + platform)
-  }
-}
-
-var platformPath = getPath(platform)
+var platformPath = getPlatformPath()
 
 if (installedVersion === version && fs.existsSync(path.join(__dirname, platformPath))) {
   process.exit(0)
@@ -61,3 +41,23 @@ function extractFile (err, zipPath) {
     })
   })
 }
+
+function onerror (err) {
+  throw err
+}
+
+function getPlatformPath () {
+  var platform = process.env.npm_config_platform || os.platform()
+
+  switch (platform) {
+    case 'darwin':
+      return 'dist/Electron.app/Contents/MacOS/Electron'
+    case 'freebsd':
+    case 'linux':
+      return 'dist/electron'
+    case 'win32':
+      return 'dist/electron.exe'
+    default:
+      throw new Error('Electron builds are not available on platform: ' + platform)
+  }
+}