On Mon, Jan 24, 2011 at 08:05:12PM +0100, Mario Camou wrote:
> 
> Gonzalo just left the company (today was his last day) and I'm taking over
> some of his Ruote responsibilities.

Hello Mario,

let's wish all the best to Gonzalo for his next endeavour.

> (...)
> 
> However, it ends up creating the ruote_launchitems queue instead of the
> ruote queue. I found the following in ruote-amqp/lib/ruote-amqp
> /receiver.rb:75-77
> 
>       @queue =
>         opts[:queue] ||
>         @launchitems == :only ? 'ruote_launchitems' : 'ruote_workitems'
> 
> which basically ignores opts[:queue]. I fixed it to:
> 
>       @queue =
>         opts[:queue] ||
>         (@launchitems == :only ? 'ruote_launchitems' : 'ruote_workitems')
> 
> which seems to work, but I'm curious as to whether this is an actual bug or
> if there's some reason for omitting the parentheses.

Many thanks, I've reproduced the issue and integrated your patch :

  
https://github.com/kennethkalmer/ruote-amqp/commit/67656246f030760af274eb6d2ed443ed3d3dd48a

This was a bug, sorry.


Best regards,

-- 
John Mettraux - http://jmettraux.wordpress.com

-- 
you received this message because you are subscribed to the "ruote users" group.
to post : send email to [email protected]
to unsubscribe : send email to [email protected]
more options : http://groups.google.com/group/openwferu-users?hl=en

Reply via email to