From: Bruce Momjian <bruce@momjian.us>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: e.indrupskaya@postgrespro.ru
Cc: pgsql-docs@lists.postgresql.org
Subject: Re: Missing directory is mentioned on page https://www.postgresql.org/docs/13/libpq-exec.html.
Date: Tue, 27 Oct 2020 14:03:19 -0400
Message-ID: <20201027180319.GQ4951@momjian.us> (raw)
In-Reply-To: <1062278.1603819969@sss.pgh.pa.us>
References: <160379609706.24746.7506163279454026608@wrigleys.postgresql.org>
<20201027164724.GN4951@momjian.us>
<1060663.1603817676@sss.pgh.pa.us>
<20201027172157.GP4951@momjian.us>
<1062278.1603819969@sss.pgh.pa.us>
On Tue, Oct 27, 2020 at 01:32:49PM -0400, Tom Lane wrote:
> Bruce Momjian <bruce@momjian.us> writes:
> > On Tue, Oct 27, 2020 at 12:54:36PM -0400, Tom Lane wrote:
> >> The removal only happened in HEAD, so I'm unclear on why you back-patched
> >> this doc change?
>
> > Uh, at the top of all my branches, I typed:
> > ls */src/tools/thread
> > and saw no hits in _any_ of the branches.
>
> Presumably this text is a leftover from before 8.2:
>
> Author: Bruce Momjian <bruce@momjian.us>
> Branch: master Release: REL8_2_BR [9e66c3e6f] 2006-02-04 01:00:02 +0000
>
> Move thread_test directory from /tools to /test so source-only tarballs
> have the directory for the configure test.
>
> > I now see that the problem
> > wasn't that /src/_tools_/thread was removed but that the docs referenced
> > it instead of src/_test_/thread. However, it doesn't compile because of
> > the pg_printf reference. I am thinking removing the paragraph in the
> > back branches was the right thing to do all along. Agreed?
>
> It probably worked okay on most platforms before v12 (96bf88d52).
> We could put back the text and fix where it pointed to, but on the
> other hand I'm not sure it's worth the trouble. Obviously nobody
> has paid any attention to that para in a long time.
I think you are right that removal is appropriate.
> >> But thanks for finding the reference; I'd only searched for test/thread,
> >> not tools/thread. Looks like src/template/netbsd has an old reference
> >> too.
>
> > Remove that too?
>
> Yeah, I think we could just nuke that line.
Removed.
--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EnterpriseDB https://enterprisedb.com
The usefulness of a cup is in its emptiness, Bruce Lee
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-docs@postgresql.org
Cc: bruce@momjian.us, tgl@sss.pgh.pa.us, e.indrupskaya@postgrespro.ru, pgsql-docs@lists.postgresql.org
Subject: Re: Missing directory is mentioned on page https://www.postgresql.org/docs/13/libpq-exec.html.
In-Reply-To: <20201027180319.GQ4951@momjian.us>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox