summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZehan Zhao <cnallenzhao@gmail.com>2016-08-16 23:35:54 +0800
committerZehan Zhao <cnallenzhao@gmail.com>2016-08-16 23:35:54 +0800
commit7b87609371cc9d2e2fa6331dae757d9d3147a671 (patch)
treef7b96d7c7385743af28abed6b5eceb468265b053
parent138b8c12c4006ec609726cf0e9902f82a5a34a87 (diff)
downloadbundler-7b87609371cc9d2e2fa6331dae757d9d3147a671.tar.gz
Add todo for #capture_and_filter_stderr
-rw-r--r--lib/bundler/source/git/git_proxy.rb5
1 files changed, 5 insertions, 0 deletions
diff --git a/lib/bundler/source/git/git_proxy.rb b/lib/bundler/source/git/git_proxy.rb
index 4b76d18735..c44f00d7b1 100644
--- a/lib/bundler/source/git/git_proxy.rb
+++ b/lib/bundler/source/git/git_proxy.rb
@@ -224,6 +224,11 @@ module Bundler
raise GitError, "The git source #{uri} is not yet checked out. Please run `bundle install` before trying to start your application"
end
+ # TODO: Replace this with Open3 when upgrading to bundler 2
+ # Similar to #git_null, as Open3 is not cross-platform,
+ # a temporary way is to use Tempfile to capture the stderr.
+ # When replacing this using Open3, make sure git_null is
+ # also replaced by Open3, so stdout and stderr all got handled properly.
def capture_and_filter_stderr(uri)
return_value, captured_err = ""
backup_stderr = STDERR.dup