[jifty-devel] Re: [Jifty-commit] r2031 - wifty/trunk/etc

Henry Baragar Henry.Baragar at instantiated.ca
Thu Nov 2 11:12:54 EST 2006


On Thursday, November 02 2006 09:29 am, Jesse Vincent wrote:
> >
> > I propose that the following lines be added to the distributed
> > etc/config.yml file:
> >
> >   SiteConfig: etc/site_config.yml
> >   TestConfig: etc/test_config.yml
> >   VendorConfig: etc/vendor_config.yml
> >
> > That way, other Jifty newbies can avoid the couple of hours I spent
> > trying to figure out if this feature is present and how to use it.
>
> Rather than hardcoding it into the generated configuration file (You may
> have guessed by now that I don't like to hardcode things that fall
> through sanely, what about adding a comment to the generated
> configuration file with a pointer to the documentation?
>

A pointer to the documentation would be good.

Otherwise, I am confused about your comments...

Regarding "fall through sanely", what is the fall through?  I had interpreted 
that to mean that it would look for a site_config.yml file, even if there is 
no SiteConfig directive in config.yml.  My first reaction was that I must 
have done something wrong, like misspelling site_config.yml. However, the 
real reason was that site_config.yml was not found until I explicitly put the 
SiteConfig directive into config.yml.  

Regarding hardcoding, what is being hardcoded?  SiteConfig is hardcoded in 
Jifty::Config.  And, nothing bad happens if etc/site_config.yml does not 
exist.  Maybe, my confusion of the hardcoding is due to my confusion about 
"fall through sanely".

Regards,
Henry

> Best,
> Jesse
>
> > Regards,
> > Henry
> >


More information about the jifty-devel mailing list