diff options
author | David RodrÃguez <deivid.rodriguez@riseup.net> | 2019-08-21 20:52:35 +0200 |
---|---|---|
committer | David RodrÃguez <deivid.rodriguez@riseup.net> | 2019-08-26 11:46:07 +0200 |
commit | e93bce3b206895a46b9fb5889c5f908fd29ad554 (patch) | |
tree | 82c547f0bb7c4848a24394ab08fb38292a7d5911 /lib | |
parent | b4cc36deb9749fba603d149e2b8e71b10cb331f1 (diff) | |
download | bundler-e93bce3b206895a46b9fb5889c5f908fd29ad554.tar.gz |
Revert "Remove now meaningless warning"
This reverts commit 00b095b98fe4bd44950beaf3bc9f1d91eac7b69e.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/bundler/definition.rb | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/lib/bundler/definition.rb b/lib/bundler/definition.rb index 08bd19e337..3f5e8ce97b 100644 --- a/lib/bundler/definition.rb +++ b/lib/bundler/definition.rb @@ -896,7 +896,17 @@ module Bundler dependencies.each do |dep| dep = Dependency.new(dep, ">= 0") unless dep.respond_to?(:name) next if !remote && !dep.current_platform? - dep.gem_platforms(sorted_platforms).each do |p| + platforms = dep.gem_platforms(sorted_platforms) + if platforms.empty? && !Bundler.settings[:disable_platform_warnings] + mapped_platforms = dep.expanded_platforms + Bundler.ui.warn \ + "The dependency #{dep} will be unused by any of the platforms Bundler is installing for. " \ + "Bundler is installing for #{@platforms.join ", "} but the dependency " \ + "is only for #{mapped_platforms.join ", "}. " \ + "To add those platforms to the bundle, " \ + "run `bundle lock --add-platform #{mapped_platforms.join " "}`." + end + platforms.each do |p| deps << DepProxy.new(dep, p) if remote || p == generic_local_platform end end |