Only fetch repositories once per installation.

gdotdesign · Aug 7, 2017 · 16c5082 · 16c5082
1 parent 44dabdf
commit 16c5082
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lib/elm_install/git_source.rb b/lib/elm_install/git_source.rb
@@ -7,6 +7,8 @@ class GitSource < Source
     # @return [Branch] The branch
     attr_reader :branch
 
+    @@fetched = {}
+
     Contract Uri, Branch => GitSource
     # Initializes a git source by URI and branch
     #
@@ -70,9 +72,10 @@ def copy_to(version, directory)
     #
     # @return [Array] The versions
     def versions(constraints)
-      if repository.cloned?
+      if repository.cloned? && !@@fetched[url]
         # Get updates from upstream
         Logger.arrow "Getting updates for: #{package_name.bold}"
+        @@fetched[url] = true
         repository.fetch
       end
 

diff --git a/lib/elm_install/resolver.rb b/lib/elm_install/resolver.rb
@@ -36,8 +36,6 @@ def resolve
     #
     # @return nil
     def resolve_dependency(dependency)
-      return if @dependencies[dependency.name]
-
       @dependencies[dependency.name] ||= dependency
 
       dependency