| From: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: commitfest 2016-11 status summary |
| Date: | 2016-11-09 02:08:30 |
| Message-ID: | CAB7nPqQbW3r0vAihqGXk8urOBsV3WKf52n4GsNT=CeyAWhHUSg@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Nov 9, 2016 at 10:53 AM, Robert Haas <robertmhaas(at)gmail(dot)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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2016-11-09 02:12:10 | Re: WIP: About CMake v2 |
| Previous Message | Robert Haas | 2016-11-09 01:53:51 | Re: commitfest 2016-11 status summary |