[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#818237: Re: vagrant-libvirt: undefined method `group_by' for nil:NilClass (NoMethodError)



Control: tag -1 pending

On Fri, Apr 01, 2016 at 02:22:14PM +0200, Christian Hofstaedtler wrote:
> This might be a suitable patch for vagrant, but I clearly don't
> understand enough of what vagrant is doing there in the first place.
> (The patch makes vagrant plugin install vagrant-libvirt work for
> me, but that's the only thing I tried.)

This seems to work for me as well. I have a locally-built package with
this patch installed here, and if nothing weird shows up I should upload
a fixed package during the weekend.

I have one question on the patch though:

> ---
>  lib/vagrant/bundler.rb | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/vagrant/bundler.rb b/lib/vagrant/bundler.rb
> index 5a5c185..c4a3837 100644
> --- a/lib/vagrant/bundler.rb
> +++ b/lib/vagrant/bundler.rb
> @@ -272,7 +272,6 @@ module Vagrant
>  
>        # 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?

-- 
Antonio Terceiro <terceiro@debian.org>

Attachment: signature.asc
Description: PGP signature


Reply to: