summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Moore <tmoore@incrementalism.net>2015-11-28 09:16:20 +1100
committerTim Moore <tmoore@incrementalism.net>2015-11-28 09:16:20 +1100
commit9e055dcbe11210edda955edb6b6722f1b2285bac (patch)
treeea151176f97b95de649c477b98e6713577f10f66
parent437f604956e629e1b96270d87f6fdb05b189de2d (diff)
downloadbundler-9e055dcbe11210edda955edb6b6722f1b2285bac.tar.gz
Explain why we don't use HTTPS for github sources
There are frequent pull requests proposing this change, which has already been merged to 2-0-dev.
-rw-r--r--lib/bundler/dsl.rb13
1 files changed, 13 insertions, 0 deletions
diff --git a/lib/bundler/dsl.rb b/lib/bundler/dsl.rb
index 4755fe0609..4ef64e1258 100644
--- a/lib/bundler/dsl.rb
+++ b/lib/bundler/dsl.rb
@@ -220,6 +220,19 @@ module Bundler
def add_git_sources
git_source(:github) do |repo_name|
+ # It would be better to use https instead of the git protocol, but this
+ # can break deployment of existing locked bundles when switching between
+ # different versions of Bundler. The change will be made in 2.0, which
+ # does not guarantee compatibility with the 1.x series.
+ #
+ # See https://github.com/bundler/bundler/pull/2569 for discussion
+ #
+ # This can be overridden by adding this code to your Gemfiles:
+ #
+ # git_source(:github) do |repo_name|
+ # repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?("/")
+ # "https://github.com/#{repo_name}.git"
+ # end
repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?("/")
"git://github.com/#{repo_name}.git"
end