ruby-changes:70103
From: David <ko1@a...>
Date: Wed, 8 Dec 2021 07:59:50 +0900 (JST)
Subject: [ruby-changes:70103] f0ef9ffed1 (master): [rubygems/rubygems] Cancel deprecation of custom git sources
https://git.ruby-lang.org/ruby.git/commit/?id=f0ef9ffed1 From f0ef9ffed1afe6a4add8959ed54b986d952c0a31 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?David=20Rodr=C3=ADguez?= <deivid.rodriguez@r...> Date: Tue, 7 Dec 2021 19:10:46 +0100 Subject: [rubygems/rubygems] Cancel deprecation of custom git sources https://github.com/rubygems/rubygems/commit/99cd6e0627 --- lib/bundler/dsl.rb | 27 ----------- spec/bundler/other/major_deprecation_spec.rb | 69 ---------------------------- 2 files changed, 96 deletions(-) diff --git a/lib/bundler/dsl.rb b/lib/bundler/dsl.rb index 1108fc3b781..f7922b1fba0 100644 --- a/lib/bundler/dsl.rb +++ b/lib/bundler/dsl.rb @@ -277,9 +277,6 @@ module Bundler https://github.com/ruby/ruby/blob/trunk/lib/bundler/dsl.rb#L277 def add_git_sources git_source(:github) do |repo_name| - warn_deprecated_git_source(:github, <<-'RUBY'.strip, 'Change any "reponame" :github sources to "username/reponame".') -"https://github.com/#{repo_name}.git" - RUBY if repo_name =~ GITHUB_PULL_REQUEST_URL { "git" => "https://github.com/#{$1}.git", @@ -294,18 +291,10 @@ module Bundler https://github.com/ruby/ruby/blob/trunk/lib/bundler/dsl.rb#L291 end git_source(:gist) do |repo_name| - warn_deprecated_git_source(:gist, '"https://gist.github.com/#{repo_name}.git"') - "https://gist.github.com/#{repo_name}.git" end git_source(:bitbucket) do |repo_name| - warn_deprecated_git_source(:bitbucket, <<-'RUBY'.strip) -user_name, repo_name = repo_name.split("/") -repo_name ||= user_name -"https://#{user_name}@bitbucket.org/#{user_name}/#{repo_name}.git" - RUBY - user_name, repo_name = repo_name.split("/") repo_name ||= user_name "https://#{user_name}@bitbucket.org/#{user_name}/#{repo_name}.git" @@ -490,22 +479,6 @@ repo_name ||= user_name https://github.com/ruby/ruby/blob/trunk/lib/bundler/dsl.rb#L479 end end - def warn_deprecated_git_source(name, replacement, additional_message = nil) - additional_message &&= " #{additional_message}" - replacement = if replacement.count("\n").zero? - "{|repo_name| #{replacement} }" - else - "do |repo_name|\n#{replacement.to_s.gsub(/^/, " ")}\n end" - end - - Bundler::SharedHelpers.major_deprecation 3, <<-EOS -The :#{name} git source is deprecated, and will be removed in the future.#{additional_message} Add this code to the top of your Gemfile to ensure it continues to work: - - git_source(:#{name}) #{replacement} - - EOS - end - class DSLError < GemfileError # @return [String] the description that should be presented to the user. # diff --git a/spec/bundler/other/major_deprecation_spec.rb b/spec/bundler/other/major_deprecation_spec.rb index b228027c03f..c6490ad2559 100644 --- a/spec/bundler/other/major_deprecation_spec.rb +++ b/spec/bundler/other/major_deprecation_spec.rb @@ -534,75 +534,6 @@ RSpec.describe "major deprecations" do https://github.com/ruby/ruby/blob/trunk/spec/bundler/other/major_deprecation_spec.rb#L534 pending "fails with a helpful error", :bundler => "3" end - describe Bundler::Dsl do - before do - @rubygems = double("rubygems") - allow(Bundler::Source::Rubygems).to receive(:new) { @rubygems } - end - - context "with github gems" do - it "does not warn about removal", :bundler => "< 3" do - expect(Bundler.ui).not_to receive(:warn) - subject.gem("sparks", :github => "indirect/sparks") - github_uri = "https://github.com/indirect/sparks.git" - expect(subject.dependencies.first.source.uri).to eq(github_uri) - end - - it "warns about removal", :bundler => "3" do - msg = <<-EOS -The :github git source is deprecated, and will be removed in the future. Change any "reponame" :github sources to "username/reponame". Add this code to the top of your Gemfile to ensure it continues to work: - - git_source(:github) {|repo_name| "https://github.com/\#{repo_name}.git" } - - EOS - expect(Bundler.ui).to receive(:warn).with("[DEPRECATED] #{msg}") - subject.gem("sparks", :github => "indirect/sparks") - github_uri = "https://github.com/indirect/sparks.git" - expect(subject.dependencies.first.source.uri).to eq(github_uri) - end - end - - context "with bitbucket gems" do - it "does not warn about removal", :bundler => "< 3" do - expect(Bundler.ui).not_to receive(:warn) - subject.gem("not-really-a-gem", :bitbucket => "mcorp/flatlab-rails") - end - - it "warns about removal", :bundler => "3" do - msg = <<-EOS -The :bitbucket git source is deprecated, and will be removed in the future. Add this code to the top of your Gemfile to ensure it continues to work: - - git_source(:bitbucket) do |repo_name| - user_name, repo_name = repo_name.split("/") - repo_name ||= user_name - "https://\#{user_name}@bitbucket.org/\#{user_name}/\#{repo_name}.git" - end - - EOS - expect(Bundler.ui).to receive(:warn).with("[DEPRECATED] #{msg}") - subject.gem("not-really-a-gem", :bitbucket => "mcorp/flatlab-rails") - end - end - - context "with gist gems" do - it "does not warn about removal", :bundler => "< 3" do - expect(Bundler.ui).not_to receive(:warn) - subject.gem("not-really-a-gem", :gist => "1234") - end - - it "warns about removal", :bundler => "3" do - msg = <<-EOS -The :gist git source is deprecated, and will be removed in the future. Add this code to the top of your Gemfile to ensure it continues to work: - - git_source(:gist) {|repo_name| "https://gist.github.com/\#{repo_name}.git" } - - EOS - expect(Bundler.ui).to receive(:warn).with("[DEPRECATED] #{msg}") - subject.gem("not-really-a-gem", :gist => "1234") - end - end - end - context "bundle show" do before do install_gemfile <<-G -- cgit v1.2.1 -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/