Commit ac04c590 authored by Marius Rackwitz's avatar Marius Rackwitz

Refactored checks for update_mode? by allowing different states through symbols with update_mode

parent 6091ea45
...@@ -103,6 +103,19 @@ module Pod ...@@ -103,6 +103,19 @@ module Pod
!!update !!update
end end
# @return [Symbol] Whether and how the dependencies in the Podfile
# should be updated.
#
def update_mode
if !update
:none
elsif update == true
:all
elsif update[:pods] != nil
:selected
end
end
# @return [Bool] Whether the analysis allows pre-downloads and thus # @return [Bool] Whether the analysis allows pre-downloads and thus
# modifications to the sandbox. # modifications to the sandbox.
# #
...@@ -225,9 +238,11 @@ module Pod ...@@ -225,9 +238,11 @@ module Pod
# that prevent the resolver to update a Pod. # that prevent the resolver to update a Pod.
# #
def generate_version_locking_dependencies def generate_version_locking_dependencies
if update_mode? # TODO: Add a case for specific pods
case update_mode
when :all
[] []
else when :none
result.podfile_state.unchanged.map do |pod| result.podfile_state.unchanged.map do |pod|
lockfile.dependency_to_lock_pod_named(pod) lockfile.dependency_to_lock_pod_named(pod)
end end
...@@ -258,9 +273,11 @@ module Pod ...@@ -258,9 +273,11 @@ module Pod
deps_to_fetch = [] deps_to_fetch = []
deps_to_fetch_if_needed = [] deps_to_fetch_if_needed = []
deps_with_external_source = podfile.dependencies.select { |dep| dep.external_source } deps_with_external_source = podfile.dependencies.select { |dep| dep.external_source }
if update_mode? # TODO: Add a case for specific pods
case update_mode
when :all
deps_to_fetch = deps_with_external_source deps_to_fetch = deps_with_external_source
else when :none
pods_to_fetch = result.podfile_state.added + result.podfile_state.changed pods_to_fetch = result.podfile_state.added + result.podfile_state.changed
deps_to_fetch = deps_with_external_source.select { |dep| pods_to_fetch.include?(dep.root_name) } deps_to_fetch = deps_with_external_source.select { |dep| pods_to_fetch.include?(dep.root_name) }
deps_to_fetch_if_needed = deps_with_external_source.select { |dep| result.podfile_state.unchanged.include?(dep.root_name) } deps_to_fetch_if_needed = deps_with_external_source.select { |dep| result.podfile_state.unchanged.include?(dep.root_name) }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment