Re: includeifexists in configuration file

On 11/16/2011 10:19 AM, Robert Haas wrote:
> I haven't read the code yet, but just to get the bikeshedding started,
> I think it might be better to call this include_if_exists rather than
> running it together as one word.
>    

What's going on, it's like this bikeshed just disappeared.  I should 
figure out how that happened so I can replicate it.

This naming style change sounds fine to me, and I just adopted it for 
the updated configuration directory patch.  That patch now rearranges 
the documentation this feature modifies.  This is a pretty trivial 
feature I'm not real concerned about getting a review for.  I'll update 
this with the name change and appropriate rebased patch once some 
decision has been made about that one; will just bounce this forward to 
January if it's still here when the current CF starts closing in earnest.

-- 
Greg Smith   2ndQuadrant US    greg@2ndQuadrant.com   Baltimore, MD
PostgreSQL Training, Services, and 24x7 Support  www.2ndQuadrant.us