Re: Using AF_UNIX sockets always for tests on Windows - Mailing list pgsql-hackers

From vignesh C
Subject Re: Using AF_UNIX sockets always for tests on Windows
Date
Msg-id CALDaNm12QffiviYms84kif2HhRnjSr_0DzjW5FzDdQzB-w6cKg@mail.gmail.com
Whole thread Raw
In response to Re: Using AF_UNIX sockets always for tests on Windows  (Thomas Munro <thomas.munro@gmail.com>)
Responses Re: Using AF_UNIX sockets always for tests on Windows
List pgsql-hackers
On Wed, 22 Mar 2023 at 09:59, Thomas Munro <thomas.munro@gmail.com> wrote:
>
> Thanks Tom, Andres, Juan José, Andrew for your feedback.  I agree that
> it's a better "OS harmonisation" outcome if we can choose both ways on
> both OSes.  I will leave the 0003 patch aside for now due to lack of
> time, but here's a rebase of the first two patches.  Since this is
> really just more cleanup-obsolete-stuff background work, I'm going to
> move it to the next CF.
>
> 0001 -- Teach mkdtemp() to care about permissions on Windows.
> Reviewed by Juan José, who confirmed my blind-coded understanding and
> showed an alternative version but didn't suggest doing it that way.
> It's a little confusing that NULL "attributes" means something
> different from NULL "descriptor", so I figured I should highlight that
> difference more clearly with some new comments.  I guess one question
> is "why should we expect the calling process to have the default
> access token?"
>
> 0002 -- Use AF_UNIX for pg_regress.  This one removes a couple of
> hundred Windows-only lines that set up SSPI stuff, and some comments
> about a signal handling hazard that -- as far as I can tell by reading
> manuals -- was never really true.
>
> 0003 -- TAP testing adjustments needs some more work based on feedback
> already given, not included in this revision.

The patch does not apply anymore:
patch -p1 < v3-0002-Always-use-AF_UNIX-sockets-in-pg_regress-on-Windo.patch
patching file src/test/regress/pg_regress.c
...
Hunk #6 FAILED at 781.
...
1 out of 10 hunks FAILED -- saving rejects to file
src/test/regress/pg_regress.c.rej

Regards,
Vignesh



pgsql-hackers by date:

Previous
From: Yongtao Huang
Date:
Subject: Fix some typos
Next
From: vignesh C
Date:
Subject: Re: some namespace.c refactoring