Improve documentation about include_dir parameter.

Started by Emanuel Araújoabout 7 years ago4 messagesdocs
Jump to latest
#1Emanuel Araújo
eacshm@gmail.com

when I choice to use conf files instead of postgresql.conf I need uncomment
parameter in postgresql.conf to work fine. It is a exception over defaults
parameters in postgresql.conf?.

Documentation don't explain that.

How can submit improve patch in documentation about that.

--

*Atenciosamente,Emanuel Araújo*

*Linux Certified, DBA PostgreSQL*

#2Emanuel Araújo
eacshm@gmail.com
In reply to: Emanuel Araújo (#1)
Re: Improve documentation about include_dir parameter.

Hi,

when I choose conf files instead of postgresql.conf I needed uncomment
"include_dir" parameter in postgresql.conf to work fine. It is a exception
over defaults parameters in postgresql.conf.

Documentation don't explain that.

How can submit improve patch in documentation about that.

--

*Atenciosamente,Emanuel Araújo*

*Linux Certified, DBA PostgreSQL*

#3Bruce Momjian
bruce@momjian.us
In reply to: Emanuel Araújo (#2)
Re: Improve documentation about include_dir parameter.

On Tue, Apr 9, 2019 at 10:23:37AM -0300, Emanuel Ara�jo wrote:

Hi,

when I choose conf files instead of postgresql.conf I needed uncomment
"include_dir" parameter in postgresql.conf to work fine.� It is a exception
over defaults parameters in postgresql.conf.

Documentation don't explain that.

How can submit improve patch in documentation about that.

I see your point. In every other case, when you see a setting and value
in postgresql.conf, the value is the _default_. However in this case:

#include_dir = 'conf.d'

'conf.d' is not the default, and you have to remove the comment to use
'conf.d' as a configuration directory.

This is not mentioned in the docs. However, I think the problem is not
the docs, but rather postgresql.conf.sample. The attached patch fixes
this, and I assume it should be backpatched, based on previous
discussions of backpatching this file.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ As you are, so once was I.  As I am, so you will be. +
+                      Ancient Roman grave inscription +

Attachments:

config.difftext/x-diff; charset=us-asciiDownload+4-4
#4Bruce Momjian
bruce@momjian.us
In reply to: Bruce Momjian (#3)
Re: Improve documentation about include_dir parameter.

On Fri, Apr 12, 2019 at 01:40:25PM -0400, Bruce Momjian wrote:

On Tue, Apr 9, 2019 at 10:23:37AM -0300, Emanuel Ara�jo wrote:

Hi,

when I choose conf files instead of postgresql.conf I needed uncomment
"include_dir" parameter in postgresql.conf to work fine.� It is a exception
over defaults parameters in postgresql.conf.

Documentation don't explain that.

How can submit improve patch in documentation about that.

I see your point. In every other case, when you see a setting and value
in postgresql.conf, the value is the _default_. However in this case:

#include_dir = 'conf.d'

'conf.d' is not the default, and you have to remove the comment to use
'conf.d' as a configuration directory.

This is not mentioned in the docs. However, I think the problem is not
the docs, but rather postgresql.conf.sample. The attached patch fixes
this, and I assume it should be backpatched, based on previous
discussions of backpatching this file.

Patch applied back through 9.4. Thanks for the report.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ As you are, so once was I.  As I am, so you will be. +
+                      Ancient Roman grave inscription +