diff options
author | Samuel Giddins <segiddins@segiddins.me> | 2016-06-24 13:47:03 -0500 |
---|---|---|
committer | Samuel Giddins <segiddins@segiddins.me> | 2016-06-24 13:47:03 -0500 |
commit | 6745b68906f2829f0a39334186a417597a7e60c4 (patch) | |
tree | 8d63f0839b1ec64351a9fa4ad764fb2fb4951867 | |
parent | b21c24ad7a8b65cd388e697266dd98a9bafc4952 (diff) | |
download | bundler-seg-skip-reresolve.tar.gz |
[Definition] Add a #change_reason printed in debugseg-skip-reresolve
-rw-r--r-- | lib/bundler/definition.rb | 25 |
1 files changed, 23 insertions, 2 deletions
diff --git a/lib/bundler/definition.rb b/lib/bundler/definition.rb index efc249be91..0305f404e3 100644 --- a/lib/bundler/definition.rb +++ b/lib/bundler/definition.rb @@ -92,7 +92,7 @@ module Bundler end @ruby_version.diff(locked_ruby_version_object) end - @unlocking ||= @unlock[:ruby] || (!@locked_ruby_version ^ !@ruby_version) + @unlocking ||= @unlock[:ruby] ||= (!@locked_ruby_version ^ !@ruby_version) current_platform = Bundler.rubygems.platforms.map {|p| generic(p) }.compact.last add_platform(current_platform) @@ -220,7 +220,7 @@ module Bundler last_resolve else # Run a resolve against the locally available gems - Bundler.ui.debug("Found changes from the lockfile, re-resolving dependencies") + Bundler.ui.debug("Found changes from the lockfile, re-resolving dependencies because #{change_reason}") last_resolve.merge Resolver.resolve(expanded_dependencies, index, source_requirements, last_resolve, ruby_version) end end @@ -462,6 +462,27 @@ module Bundler !@source_changes && !@dependency_changes && !@new_platform && !@path_changes && !@local_changes end + def change_reason + if @unlocking + unlock_reason = @unlock.reject {|_k, v| Array(v).empty? }.map do |k, v| + if v == true + k.to_s + else + v = Array(v) + "#{k}: (#{v.join(", ")})" + end + end.join(", ") + return "bundler is unlocking #{unlock_reason}" + end + [ + [@source_changes, "the list of sources changed"], + [@dependency_changes, "the dependencies in your gemfile changed"], + [@new_platform, "you added a new platform to your gemfile"], + [@path_changes, "the gemspecs for path gems changed"], + [@local_changes, "the gemspecs for git local gems changed"], + ].select(&:first).map(&:last).join(", ") + end + def pretty_dep(dep, source = false) msg = String.new(dep.name) msg << " (#{dep.requirement})" unless dep.requirement == Gem::Requirement.default |