From: | Noah Misch <noah(at)leadboat(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz>, Daniel Gustafsson <daniel(at)yesql(dot)se> |
Cc: | Jacob Champion <pchampion(at)vmware(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: testclient.exe installed under MSVC |
Date: | 2022-05-03 02:02:01 |
Message-ID: | 20220503020201.GA1618915@rfd.leadboat.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, May 02, 2022 at 03:14:50PM +0200, Daniel Gustafsson wrote:
> > On 1 May 2022, at 15:23, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> > On Sun, May 01, 2022 at 01:07:06AM -0700, Noah Misch wrote:
> >> My annual audit for executables missing Windows icons turned up these:
> >>
> >> pginstall/bin/testclient.exe
> >> pginstall/bin/uri-regress.exe
> >>
> >> I was going to add the icons, but I felt the testclient.exe name is too
> >> generic-sounding to be installed. testclient originated in commit ebc8b7d. I
> >> recommend ceasing to install both programs under MSVC. (The GNU make build
> >> system does not install them.)
> >
> > But MSVC works differently. vcregress.pl does a TempInstall(), which
> > is a simple Install(), so isn't it going to be an issue for the tests
> > if these two tools are not installed anymore?
Resolving that would be part of any project to stop installing them.
> >> If that's unwanted for some reason, could you
> >> rename testclient to something like libpq_test?
> >
> > Yes, the renaming makes sense. I'd say to do more, and also rename
> > uri-regress, removing the hyphen from the binary name and prefix both
> > binaries with a "pg_".
>
> Renaming is probably the best option given how MSVC works. Using a pg_ prefix
> makes them sound like actual useful tools though with (albeit small) risk for
> confusion? Noah's suggestion of libpq_ is perhaps better: libpq_testclient.
Agreed. libpq_testclient and libpq_uri_regress sound fine.
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2022-05-03 02:13:18 | Re: strange slow query - lost lot of time somewhere |
Previous Message | Andres Freund | 2022-05-03 01:48:03 | Re: failures in t/031_recovery_conflict.pl on CI |