fix(git): only use longpaths on win32 because old gits

zkat · May 5, 2017 · 32846fc · 32846fc
1 parent 8818f96
commit 32846fc
Showing 1 changed file with 9 additions and 25 deletions.
diff --git a/lib/util/git.js b/lib/util/git.js
@@ -50,14 +50,10 @@ try {
 module.exports.clone = fullClone
 function fullClone (repo, committish, target, opts) {
   opts = optCheck(opts)
-  const gitArgs = [
-    'clone',
-    '-q',
-    // Mainly for windows, but no harm done
-    '-c', 'core.longpaths=true',
-    repo,
-    target
-  ]
+  const gitArgs = ['clone', '-q', repo, target]
+  if (process.platform === 'win32') {
+    gitArgs.push('--config', 'core.longpaths=true')
+  }
   return execGit(gitArgs, {
     cwd: path.dirname(target)
   }, opts).then(() => {
@@ -72,16 +68,10 @@ function fullClone (repo, committish, target, opts) {
 module.exports.shallow = shallowClone
 function shallowClone (repo, branch, target, opts) {
   opts = optCheck(opts)
-  const gitArgs = [
-    'clone',
-    '--depth=1',
-    '-q',
-    '-b', branch,
-    // Mainly for windows, but no harm done
-    '-c', 'core.longpaths=true',
-    repo,
-    target
-  ]
+  const gitArgs = ['clone', '--depth=1', '-q', '-b', branch, repo, target]
+  if (process.platform === 'win32') {
+    gitArgs.push('--config', 'core.longpaths=true')
+  }
   return execGit(gitArgs, {
     cwd: target
   }, opts).then(() => {
@@ -90,13 +80,7 @@ function shallowClone (repo, branch, target, opts) {
 }
 
 function updateSubmodules (localRepo, opts) {
-  const gitArgs = [
-    'submodule',
-    'update',
-    '-q',
-    '--init',
-    '--recursive'
-  ]
+  const gitArgs = ['submodule', 'update', '-q', '--init', '--recursive']
   return execGit(gitArgs, {
     cwd: localRepo
   }, opts)