Merge #6999

6999: Fix multiple gem files deprecation r=deivid-rodriguez a=deivid-rodriguez

### What was the end-user problem that led to this PR?

The problem was the current message about deprecating `Gemfile` in favor of `gems.rb` was incorrect, and this I'm not sure when it was supposed to be displayed. Its corresponding test was also failing.

### What was your diagnosis of the problem?

My diagnosis was that the intention was not to deprecate Gemfile's for the time being, but only warn if the user opts in to `gems.rb` but does not remove the old `Gemfile`.

### What is your fix for the problem, implemented in this PR?

My fix is to properly detect the situation where both `Gemfile` and `gems.rb` files are present, and show proper messages guiding the user to properly make the switch.

### Why did you choose this fix out of the possible options?

I chose this fix because I think it works and doesn't take this migration too far, since the ecosystem is not yet ready for this at all.


Co-authored-by: David Rodríguez <deivid.rodriguez@riseup.net>

1 files changed, 9 insertions, 4 deletions

diff --git a/lib/bundler/shared_helpers.rb b/lib/bundler/shared_helpers.rb
index 996f7b3fd4..0f4f54dced 100644
--- a/lib/bundler/shared_helpers.rb
+++ b/lib/bundler/shared_helpers.rb
@@ -157,11 +157,16 @@ module Bundler
       multiple_gemfiles = search_up(".") do |dir|
         gemfiles = gemfile_names.select {|gf| File.file? File.expand_path(gf, dir) }
         next if gemfiles.empty?
-        break false if gemfiles.size == 1
+        break gemfiles.size != 1
       end
-      return unless multiple_gemfiles && Bundler.bundler_major_version == 1
-      Bundler::SharedHelpers.major_deprecation 2, \
-        "gems.rb and gems.locked will be preferred to Gemfile and Gemfile.lock."
+      return unless multiple_gemfiles
+      diagnosis = "Multiple gemfiles (gems.rb and Gemfile) detected."
+      advice = if Bundler.feature_flag.prefer_gems_rb?
+        "Make sure you remove Gemfile and Gemfile.lock since bundler is ignoring them in favor of gems.rb and gems.rb.locked."
+      else
+        "The gems.rb and gems.rb.locked files are currently ignored, but they will get used as soon as you delete your Gemfile and Gemfile.lock files."
+      end
+      Bundler.ui.warn [diagnosis, advice].join(" ")
     end
 
     def trap(signal, override = false, &block)