Add a few comments on vendored dependencies

1 files changed, 14 insertions, 0 deletions

diff --git a/Rakefile b/Rakefile
index 983766850e..1c3958c9ea 100644
--- a/Rakefile
+++ b/Rakefile
@@ -264,6 +264,9 @@ else
     lib.vendor_lib = "lib/bundler/vendor/molinillo"
   end
 
+  # We currently cherry-pick changes to use `require_relative` internally
+  # instead of regular `require`. They are already in thor's master branch but
+  # still need to be released.
   desc "Vendor a specific version of thor"
   Automatiek::RakeTask.new("thor") do |lib|
     lib.download = { :github => "https://github.com/erikhuda/thor" }
@@ -272,6 +275,9 @@ else
     lib.vendor_lib = "lib/bundler/vendor/thor"
   end
 
+  # We currently cherry-pick changes to use `require_relative` internally
+  # instead of regular `require`. They are already in fileutils' master branch
+  # but still need to be released.
   desc "Vendor a specific version of fileutils"
   Automatiek::RakeTask.new("fileutils") do |lib|
     lib.download = { :github => "https://github.com/ruby/fileutils" }
@@ -280,6 +286,14 @@ else
     lib.vendor_lib = "lib/bundler/vendor/fileutils"
   end
 
+  # Currently `net-http-persistent` and it's dependency `connection_pool` are
+  # vendored separately, but `connection_pool` references inside the vendored
+  # copy of `net-http-persistent` are not properly updated to refer to the
+  # vendored copy of `connection_pool`, so they need to be manually updated.
+  # This will be automated once https://github.com/segiddins/automatiek/pull/3
+  # is included in `automatiek` and we start using the new API for vendoring
+  # subdependencies.
+
   desc "Vendor a specific version of net-http-persistent"
   Automatiek::RakeTask.new("net-http-persistent") do |lib|
     lib.download = { :github => "https://github.com/drbrain/net-http-persistent" }