fix(install): fix the logic to get the hash at git resolver

watilde · Jul 31, 2017 · e2ceb04 · e2ceb04
1 parent 6113236
commit e2ceb04
Showing 1 changed file with 4 additions and 9 deletions.
diff --git a/lib/install/resolver/fetchers/git.js b/lib/install/resolver/fetchers/git.js
@@ -42,26 +42,21 @@ module.exports = (name, spec, result) => {
         if (semver.satisfies(version, range)) hosted.committish = hash
       })
     } else {
-      console.log(git.sync(['ls-remote', httpstemplate(hosted)])
-        .split('\n')
-        .filter((str, i) => {
-          return str.indexOf(hosted.committish) !== -1
-        }))
-
-      hosted.committish = git.sync(['ls-remote', httpstemplate(hosted)])
+      const committish = git.sync(['ls-remote', '--tags', httpstemplate(hosted)])
         .split('\n')
         .filter((str, i) => {
           return str.indexOf(hosted.committish) !== -1
         }).pop()
-        .split('\t')[0]
+      hosted.committish = committish
+        ? committish.split('\t')[0]
+        : hosted.committish
     }
     const options = {
       url: filetemplate(hosted),
       headers: {
         'User-Agent': npmrc.userAgent
       }
     }
-    console.log(filetemplate(hosted))
     var body = ''
     request.get(options)
       .on('data', (chunk) => { body += chunk })