| 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
| 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 |