On Jul 17, 2009, at 4:32 PM, Christian wrote: > > > > 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?
Hrm, err, yeah. On my first reading of this I thought it was adding puppet's library directory, rather than the directory you might have puppet itself in. BTW, $:.unshift is == $:.insert(0) -- A cult is a religion with no political power. -- Tom Wolfe --------------------------------------------------------------------- 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 -~----------~----~----~----~------~----~------~--~---
