diff options
| author | The Bundler Bot <bot@bundler.io> | 2018-08-15 02:47:39 +0000 |
|---|---|---|
| committer | The Bundler Bot <bot@bundler.io> | 2018-08-15 02:47:39 +0000 |
| commit | fc60fe4362a23254602fd4082a8d63ec5daf8106 (patch) | |
| tree | 29527f8e81e337cf4f3357df01b1566188d83b25 | |
| parent | 33166ba1c8cc95f7585bc6e8d594014dc6dc8d29 (diff) | |
| parent | 750012c382b90b44ce44937a759f562e1b340725 (diff) | |
| download | bundler-fc60fe4362a23254602fd4082a8d63ec5daf8106.tar.gz | |
Auto merge of #6650 - greysteil:dont-mutate-original-trees, r=segiddins
Don't mutate original error trees when determining version_conflict_message
*I know I'm messing with private APIs here, so have no right to expect this PR merged!*
Currently, when Bundler generates its error message for a version conflict, it mutates the `requirement_trees` array on the underlying `Molinillo::VersionConflict` error.
I would really like it if it didn't. Dependabot taps into that error to understand what went wrong with resolution and unlock any dependencies that are blocking resolution. The code where we do so is [here](https://github.com/dependabot/dependabot-core/blob/9abcb85ba69b408b11b56572dd13ab89d0c55b8c/lib/dependabot/file_updaters/ruby/bundler/lockfile_updater.rb#L142-L167).
I don't think that there's any downside to not mutating here. Resolution has finished (and failed) at this stage, so trying to reduce memory usage doesn't really matter.
🙏
| -rw-r--r-- | lib/bundler/resolver.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/bundler/resolver.rb b/lib/bundler/resolver.rb index 3fd1706371..a6c26ffc6b 100644 --- a/lib/bundler/resolver.rb +++ b/lib/bundler/resolver.rb @@ -317,7 +317,7 @@ module Bundler :possibility_type => possibility_type, :reduce_trees => lambda do |trees| # called first, because we want to reduce the amount of work required to find maximal empty sets - trees.uniq! {|t| t.flatten.map {|dep| [dep.name, dep.requirement] } } + trees = trees.uniq {|t| t.flatten.map {|dep| [dep.name, dep.requirement] } } # bail out if tree size is too big for Array#combination to make any sense return trees if trees.size > 15 |