* Antonio Terceiro <[email protected]> [160401 16:34]: > On Fri, Apr 01, 2016 at 02:22:14PM +0200, Christian Hofstaedtler wrote: > > ...
> I have one question on the patch though: > > > diff --git a/lib/vagrant/bundler.rb b/lib/vagrant/bundler.rb [..] > > # Reset the all specs override that Bundler does > > old_all = Gem::Specification._all > > - Gem::Specification.all = nil > > > > # /etc/gemrc and so on. > > old_config = nil > > @@ -286,6 +285,8 @@ module Vagrant > > end > > Gem.configuration = NilGemConfig.new > > > > + Gem::Specification.reset > > + > > any specific reason to put this call down here, instead of exactly in > the place of the broken one? I -guessed- that Gem.configuration = NilGemConfig.new may reset Gem.(pre|post)_reset_hooks, but upon closer inspection it doesn't appear to do that. As such, no good reason to move it. -- ,''`. Christian Hofstaedtler <[email protected]> : :' : Debian Developer `. `' 7D1A CFFA D9E0 806C 9C4C D392 5C13 D6DB 9305 2E03 `-
