[chef] Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: segfault with CentOS 5.4/5.5 + ruby 1.8.7 + chef 0.10.0


Chronological Thread 
  • From: Sergio Rubio < >
  • To:
  • Subject: [chef] Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: Re: segfault with CentOS 5.4/5.5 + ruby 1.8.7 + chef 0.10.0
  • Date: Thu, 9 Jun 2011 18:07:38 +0200
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:content-type; b=dmOx9Vn99qwofH0jSdfB8pzaRuI+ADsE97Lnl1jQqReRMUFs7Vrc8MjXwwQSsSuAyw M8A9x+QPND3KOCEmJ6rBt0j5dFa1CBrCdGDsf9ylM5IUAvHyD4LACCk11mwHLU5b4SeT aM6rvbDWgUwtNwDuXbeB/Jnr9ZstiM8LtxeCk=

On Thu, Jun 9, 2011 at 5:36 PM, Daniel DeLeo < "> > wrote:

On Thursday, June 9, 2011 at 8:32 AM, Sergio Rubio wrote:

> If you guys feel like both approaches are valid, I can add a patch to the Chef RPM to alleviate some of the pain the CentOS users are dealing with.
>
> Rgds.
Yeah, that would be great. I'm in favor of switching popen4 to shell_out if you can since the API is cleaner and it has built-in support for nice error messages when a command fails as well as live updating of output to a tty in some conditions.

Feel free to hop on to #chef-hacking on freenode.net (http://freenode.net) if you have development questions, or you can mail the chef-dev list as well.

Alright. I've modified the patch to use shell_out:


And the full yum.rb file in case anyone wants to test it:


(replaces /usr/lib/ruby/gems/1.8/gems/chef-0.10.0/lib/chef/provider/package/yum.rb)

My plan is to do some more testing and If everything goes well, I'll push an updated RPM soon after that.

I've seen that there are a lot of changes coming to the yum provider in the next release, so I guess submitting a pull request is not necessary, right?

Rgds.


 
--
Dan DeLeo
> >  --
> >  Dan DeLeo





Archive powered by MHonArc 2.6.16.

§