| From: | Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com> |
|---|---|
| To: | kuroda(dot)hayato(at)fujitsu(dot)com |
| Cc: | michael(at)paquier(dot)xyz, pgsql-hackers(at)lists(dot)postgresql(dot)org |
| Subject: | Re: pg_ctl start may return 0 even if the postmaster has been already started on Windows |
| Date: | 2023-09-20 05:18:41 |
| Message-ID: | 20230920.141841.738987020573226748.horikyota.ntt@gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
At Tue, 19 Sep 2023 13:48:55 +0000, "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com> wrote in
> Dear Horiguchi-san,
>
> > I added the thread to next CF entry, so let's see the how cfbot says.
>
> At least there are several compiler warnings. E.g.,
>
> * pgwin32_find_postmaster_pid() has "return;", but IIUC it should be "exit(1)"
> * When DWORD is printed, "%lx" should be used.
> * The variable "flags" seems not needed.
Yeah, I thought that they all have been fixed but.. you are right in
every respect.
> Here is a patch which suppresses warnings, whereas test would fail...
> You can use it if acceptable.
I was able to see the trouble in the CI environment, but not
locally. I'll delve deeper into this. Thanks you for bringing it to my
attention.
regards.
--
Kyotaro Horiguchi
NTT Open Source Software Center
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Hayato Kuroda (Fujitsu) | 2023-09-20 05:30:11 | RE: [PoC] pg_upgrade: allow to upgrade publisher node |
| Previous Message | Ryoga Yoshida | 2023-09-20 05:08:34 | Re: Bug fix for psql's meta-command \ev |