General discussion about Chef

[chef] Re: Re: "knife cookbook site install" fails. Knife or Git issue?


Chronological Thread 
  • From: "Mark J. Reed" < >
  • To:
  • Subject: [chef] Re: Re: "knife cookbook site install" fails. Knife or Git issue?
  • Date: Fri, 6 May 2011 09:59:55 -0400
  • Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; b=wmN3V4Uh/4m6UrhJCplpor+Hmf3yfPBa+b7eumvyyKXXdLy+9WacCXzX+XKdXlJZ1V CflIhOrw+cS6uvIvw/hvrPDT4ULZTIiZ2QNRSbQOj2hIajyBm/Ypoqywlr3x6nT6mFbm ex+x3yiX83A5pDgxyzdOmk1mfhrGkLlm8FGQU=
I don't have the Chef server code in front of me, but are you sure it's calling include? on the individual strings rather than the whole list?The latter would seem to make more sense in this instance.  

["chef-vendor-rabbitmq_chef"].include?("chef-vendor-rabbitmq") => false

On Fri, May 6, 2011 at 9:12 AM, Ringo De Smet < "> > wrote:
Gents,

This is a bug in Chef, more specifically in CookbookSCMRepo.branch_exists?

In my case, I already had branch chef-vendor-rabbitmq_chef in my repo,
but now I wanted to add branch chef-vendor-rabbitmq. The
implementation checks the list of existing branches and matches using
the String.include? method, however:

"chef-vendor-rabbitmq_chef".include?("chef-vendor-rabbitmq")  => true

Filed this ticket: CHEF-2314

Ringo



--
Mark J. Reed < "> >


Archive powered by MHonArc 2.6.16.

§