Обсуждение: postgresql.conf.sample tab width

I noticed that GitHub is using 4-space tabs for this file [0], presumably
due to this [1] recent change.  I'm not aware of an official policy for
this file, but it seems to be written for 8-space tabs.

One way to fix this is to set our default tab width to 8 in
.gitattributes/.editorconfig (see attached patch).  AFAICT that doesn't
affect source code files.  We could also add a special rule for
postgresql.conf.sample, or we could even convert it to 4-space tabs.  Any
druthers?

[0] https://github.com/postgres/postgres/blob/master/src/backend/utils/misc/postgresql.conf.sample
[1] https://github.blog/changelog/2025-08-07-default-tab-size-changed-from-eight-to-four/

-- 
nathan

Nathan Bossart <nathandbossart@gmail.com> writes:
> I noticed that GitHub is using 4-space tabs for this file [0], presumably
> due to this [1] recent change.  I'm not aware of an official policy for
> this file, but it seems to be written for 8-space tabs.

> One way to fix this is to set our default tab width to 8 in
> .gitattributes/.editorconfig (see attached patch).  AFAICT that doesn't
> affect source code files.  We could also add a special rule for
> postgresql.conf.sample, or we could even convert it to 4-space tabs.  Any
> druthers?

We've been around on this topic before.  It's hard for us to deal with
8-space tabs within our repo, but we should expect that users who are
editing their postgresql.conf files are probably going to be using
8-space tabs.

I seem to recall a proposal that the best fix is to not use tabs
at all in postgresql.conf.sample.  That seems like a good way out
to me.  I think we could fix .gitattributes so that "git diff --check"
would verify no tabs, but I've not looked into details.

            regards, tom lane

On Fri, Nov 14, 2025 at 03:25:15PM -0500, Tom Lane wrote:
> I seem to recall a proposal that the best fix is to not use tabs
> at all in postgresql.conf.sample.  That seems like a good way out
> to me.  I think we could fix .gitattributes so that "git diff --check"
> would verify no tabs, but I've not looked into details.

I don't see a way in .gitattributes to check for _any_ tabs in a file, just
for beginning-of-line indentation with tabs.  But it's easy enough to add a
check in the TAP test.

-- 
nathan

Nathan Bossart <nathandbossart@gmail.com> writes:
> I don't see a way in .gitattributes to check for _any_ tabs in a file, just
> for beginning-of-line indentation with tabs.  But it's easy enough to add a
> check in the TAP test.

Works for me.  Do we need any special .gitattributes for the file
at all, if we're relying on the TAP test to catch this?

            regards, tom lane

On Fri, Nov 14, 2025 at 04:34:08PM -0500, Tom Lane wrote:
> Works for me.  Do we need any special .gitattributes for the file
> at all, if we're relying on the TAP test to catch this?

We'd at least need -whitespace (I think), but I figured we might as well
highlight beginning-of-line tabs and trailing whitespace in diffs, too.
The TAP test wouldn't catch the latter.

-- 
nathan