On Wed, Nov 9, 2016 at 10:53 AM, Robert Haas <robertmhaas@gmail.com> wrote:
> Well, then it should be marked "Waiting on Author" or "Needs Review",
> not "Ready for Committer".
>
> Or maybe just "Returned with Feedback", if the reporter hasn't poked
> his head up for a few weeks.
Since yesterday with the help of Tsunakawa-san, we found a simple way
to test things by patching pg_ctl.c so "Ready for Committer"
definitely applies now. This has allowed us to draft patches that fix
the original issue as well as another issue we bumped on the way:
Postgres being started by the Local System user does not ensure that
it is a service. The instance may have been started by another service
expecting Postgres to report to stderr.
There is a refactoring patch that simplifies the code of
win32security.c for HEAD, as well as a simple back-patchable fix for
other branches.
--
Michael