hsbt	2018-12-11 20:02:41 +0900 (Tue, 11 Dec 2018)

  New Revision: 66338

  https://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=revision&revision=66338

  Log:
    Merge Bundler 1.17.2 from upstream.

  Modified files:
    trunk/lib/bundler/dsl.rb
    trunk/lib/bundler/feature_flag.rb
    trunk/lib/bundler/version.rb
    trunk/spec/bundler/bundler/dsl_spec.rb
Index: lib/bundler/dsl.rb
===================================================================
--- lib/bundler/dsl.rb	(revision 66337)
+++ lib/bundler/dsl.rb	(revision 66338)
@@ -305,7 +305,7 @@ module Bundler https://github.com/ruby/ruby/blob/trunk/lib/bundler/dsl.rb#L305
         #   end
         repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?("/")
         # TODO: 2.0 upgrade this setting to the default
-        if Bundler.settings["github.https"]
+        if Bundler.feature_flag.github_https?
           Bundler::SharedHelpers.major_deprecation 2, "The `github.https` setting will be removed"
           "https://github.com/#{repo_name}.git"
         else
Index: lib/bundler/feature_flag.rb
===================================================================
--- lib/bundler/feature_flag.rb	(revision 66337)
+++ lib/bundler/feature_flag.rb	(revision 66338)
@@ -60,6 +60,8 @@ module Bundler https://github.com/ruby/ruby/blob/trunk/lib/bundler/feature_flag.rb#L60
 
     settings_option(:default_cli_command) { bundler_2_mode? ? :cli_help : :install }
 
+    settings_method(:github_https?, "github.https") { bundler_2_mode? }
+
     def initialize(bundler_version)
       @bundler_version = Gem::Version.create(bundler_version)
     end
Index: lib/bundler/version.rb
===================================================================
--- lib/bundler/version.rb	(revision 66337)
+++ lib/bundler/version.rb	(revision 66338)
@@ -7,7 +7,7 @@ module Bundler https://github.com/ruby/ruby/blob/trunk/lib/bundler/version.rb#L7
   # We're doing this because we might write tests that deal
   # with other versions of bundler and we are unsure how to
   # handle this better.
-  VERSION = "1.17.1" unless defined?(::Bundler::VERSION)
+  VERSION = "1.17.2" unless defined?(::Bundler::VERSION)
 
   def self.overwrite_loaded_gem_version
     begin
Index: spec/bundler/bundler/dsl_spec.rb
===================================================================
--- spec/bundler/bundler/dsl_spec.rb	(revision 66337)
+++ spec/bundler/bundler/dsl_spec.rb	(revision 66338)
@@ -25,7 +25,23 @@ RSpec.describe Bundler::Dsl do https://github.com/ruby/ruby/blob/trunk/spec/bundler/bundler/dsl_spec.rb#L25
       expect { subject.git_source(:example) }.to raise_error(Bundler::InvalidOption)
     end
 
+    context "github_https feature flag" do
+      it "is true when github.https is true" do
+        bundle "config github.https true"
+        expect(Bundler.feature_flag.github_https?).to eq "true"
+      end
+    end
+
     context "default hosts (git, gist)", :bundler => "< 2" do
+      context "when github.https config is true" do
+        before { bundle "config github.https true" }
+        it "converts :github to :git using https" do
+          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
+
       it "converts :github to :git" do
         subject.gem("sparks", :github => "indirect/sparks")
         github_uri = "git://github.com/indirect/sparks.git"

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/