Re: BUG #13594: pg_ctl.exe redirects stderr to Windows Events Log if stderr is redirected to pipe

On Thu, Sep 3, 2015 at 9:28 PM, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
wrote:

> On Thu, Sep 3, 2015 at 8:36 PM, Andres Freund <andres(at)anarazel(dot)de> wrote:
> > On 2015-08-31 15:51:57 +0900, Michael Paquier wrote:
> >> if (!OpenProcessToken(GetCurrentProcess(), TOKEN_READ,
> &AccessToken))
> >> {
> >> +#ifndef FRONTEND
> >> write_stderr("could not open process token: error code
> %lu\n",
> >> GetLastError());
> >> +#else
> >> + fprintf(stderr, "could not open process token: error code
> %lu\n",
> >> + GetLastError());
> >> +#endif
> >> exit(1);
> >> }
> >
> > I find these kind of ifdefs rather ugly - why not just introduce a
> > wrapper?
>
> I thought it was just not worth it for this file. I don't mind
> updating if you think that's cleaner this way.
>

New patch attached, updated in consequence.
--
Michael