Message-ID: <a4f3aa1a03eec651be55a6bc68b82a29f3080a16.camel@cybertec.at>
Subject: Re: Not an error but a difficult wording
From: Laurenz Albe <laurenz.albe@cybertec.at>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: riivo.kolka@gmail.com, pgsql-docs@lists.postgresql.org
Date: Thu, 26 Jan 2023 05:49:47 +0100
In-Reply-To: <2873117.1674697141@sss.pgh.pa.us>
References: 
	<167463493588.2667301.13267758265445155872@wrigleys.postgresql.org>
	 <cd3c9ee208ccf0cc37b5e9ce3c24f79bc229a600.camel@cybertec.at>
	 <2873117.1674697141@sss.pgh.pa.us>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
User-Agent: Evolution 3.46.3 (3.46.3-1.fc37) 
MIME-Version: 1.0
Archived-At: 
 <https://www.postgresql.org/message-id/a4f3aa1a03eec651be55a6bc68b82a29f3080a16.camel%40cybertec.at>
Precedence: bulk

On Wed, 2023-01-25 at 20:39 -0500, Tom Lane wrote:
> Laurenz Albe <laurenz.albe@cybertec.at> writes:
> > On Wed, 2023-01-25 at 08:22 +0000, PG Doc comments form wrote:
> > > Maybe this would be better? (I don't know the comma rules)
> > > "because the files(,?) that are generated/processed by these tools(,?=
) are
> > > already included in the tarball"
>=20
> > +1
>=20
> > Correct English would be:
>=20
> > =C2=A0 These tools are not needed to build from a distribution tarball,=
 because
> > =C2=A0 the files generated by these tools are included in the tarball.
>=20
> The existing wording is not incorrect AFAICS, but I agree it's a bit
> awkward.

I meant "a correct version of what was suggested in the mail", not that
the released text was incorrect.

> I'd modify one word in your version:
>=20
> =C2=A0 These tools are not needed to build from a distribution tarball, b=
ecause
> =C2=A0 the files generated using these tools are included in the tarball.
>=20
> Or possibly "with" instead of "using"?

Both are better; I'd lean towards "with".

Yours,
Laurenz Albe