Re: Add additional information to src/test/ssl/README

From: Kevin Burke <kevin(at)burke(dot)dev>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Add additional information to src/test/ssl/README
Date: 2021-10-30 17:46:09
Message-ID: CAKcy5ei6TjGCz0mL9ZQNT=J4WgqNF5OJqYEZFqmtcEfrY553Sw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

I probably would not have looked there honestly; I was working in the
terminal and had the source code right there. Could we put a link to that
page in the README?

"For more information on Postgres's TAP tests, see the docs:
https://www.postgresql.org/docs/devel/regress-tap.html"

Kevin

On Sat, Oct 30, 2021 at 7:14 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> Kevin Burke <kevin(at)burke(dot)dev> writes:
> > I've been trying to run the SSL tests against my branch and that was
> > tougher than expected because I didn't realize that additional output was
> > being saved that I couldn't see - it wasn't even getting to the part
> where
> > it could run the tests. This patch adds a note to the README explaining
> > where to look if you get an error.
>
> That's a generic thing about every one of our TAP tests, so documenting
> it in that one README doesn't seem like an appropriate answer. Would
> it have helped you to explain it in
>
> https://www.postgresql.org/docs/devel/regress-tap.html
>
> ?
>
> regards, tom lane
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2021-10-30 18:12:53 Re: Add additional information to src/test/ssl/README
Previous Message Tom Lane 2021-10-30 14:14:46 Re: Add additional information to src/test/ssl/README