Re: postgresql.conf.sample tab width

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: postgresql.conf.sample tab width
Date: 2025-11-14 20:25:15
Message-ID: 3230808.1763151915@sss.pgh.pa.us
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Nathan Bossart <nathandbossart(at)gmail(dot)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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Sami Imseih 2025-11-14 20:40:36 Have BackendXidGetPid return pid_t
Previous Message Masahiko Sawada 2025-11-14 20:19:47 Re: Make COPY format extendable: Extract COPY TO format implementations