Re: [PATCH] Add ACL (Access Control List) acronym - Mailing list pgsql-hackers

From Nathan Bossart
Subject Re: [PATCH] Add ACL (Access Control List) acronym
Date
Msg-id Znw3_THbGHgBQvQf@nathan
Whole thread Raw
In response to Re: [PATCH] Add ACL (Access Control List) acronym  ("David G. Johnston" <david.g.johnston@gmail.com>)
Responses Re: [PATCH] Add ACL (Access Control List) acronym
List pgsql-hackers
On Wed, Jun 26, 2024 at 07:58:55AM -0700, David G. Johnston wrote:
> On Wed, Jun 26, 2024 at 7:52 AM Joel Jacobson <joel@compiler.org> wrote:
>> Want me to fix that or will the committer handle that?
>>
>> I found some more similar cases in acronyms.sgml.
>
> Given this I'd be OK with committing as-is in the name of matching existing
> project style.  Then bringing up this inconsistency as a separate concern
> to be bulk fixed as part of implementing a new policy on what to check for
> and conform to when establishing acronyms in our documentation.
> 
> Otherwise the author (you) should make the change here - the committer
> wouldn't be expected to know to do that from the discussion.

If I was writing these patches, I'd create a separate 0001 patch to fix the
existing problems, then 0002 would be just the new stuff (without the
inconsistency).  But that's just what I'd do; there's no problem with doing
it the other way around.

-- 
nathan



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: pgindent exit status if a file encounters an error
Next
From: Nathan Bossart
Date:
Subject: Re: improve predefined roles documentation