On 18.07.2009, at 00:30, Luke Kanies <[email protected]> wrote:

>
> On Jul 17, 2009, at 1:11 PM, Marc Fournier wrote:
>
>>
>> If you happen to already have puppet code somewhere in your ruby
>> path, you
>> might get unexpected behaviour when trying to run puppetmaster with
>> passenger.
>>
>> Suggesting this instead could save time to some.
>>
>> Signed-off-by: Marc Fournier <[email protected]>
>> ---
>> ext/rack/files/config.ru |    2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/ext/rack/files/config.ru b/ext/rack/files/config.ru
>> index 5607bab..e7fd627 100644
>> --- a/ext/rack/files/config.ru
>> +++ b/ext/rack/files/config.ru
>> @@ -2,7 +2,7 @@
>> # SSL needs to be handled outside this, though.
>>
>> # if puppet is not in your RUBYLIB:
>> -# $:.push('/opt/puppet/lib')
>> +# $:.insert(0, '/opt/puppet/lib')
>
>
> Shouldn't this require puppet and then push Puppet[:libdir]?
>
> I like the patch, but we might as well make it better in general,  
> right

Can't imagine how we could require puppet before fixing the  
searchpath...
Maybe I'm missing something here?

-ch

> -- 
> While one person hesitates because he feels inferior, the other is
> busy making mistakes and becoming superior. -- Henry C. Link
> ---------------------------------------------------------------------
> Luke Kanies | http://reductivelabs.com | http://madstop.com
>
>
> >

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Puppet Developers" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/puppet-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to