deltacloud-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Lutterkort <lut...@redhat.com>
Subject Re: [PATCH core 1/4] Core: Added 'lib/initializers' directory
Date Sat, 16 Feb 2013 00:12:33 GMT
On Fri, 2013-02-15 at 11:36 +0100, mfojtik@redhat.com wrote:
> From: Michal Fojtik <mfojtik@redhat.com>
> 
> Before this patch, we used to initialize stuff like database
> or mock directories in random places in source code.
> 
> This patch should make all initialization happen in one place.

ACK.  Bringing some order into the initialization process is long
overdue ...

> diff --git a/server/config.ru b/server/config.ru
> index 406c769..305ab4b 100644
> --- a/server/config.ru
> +++ b/server/config.ru
...

> +load File.join(File.dirname(__FILE__), 'lib', 'initialize.rb')

Why load ? Do we ever want to run initialization more than once ? If
not, require_relative would be enough.

David



Mime
View raw message