Patch BUG #5103: "pg_ctl -w (re)start" fails with custom unix_socket_directory

Started by Quan Zongliangover 15 years ago6 messageshackers
Jump to latest
#1Quan Zongliang
quanzongliang@gmail.com

Hi, all

I created a pg_ctl patch to fix:
* BUG #5103: "pg_ctl -w (re)start" fails with custom unix_socket_directory
Allow pg_ctl to work properly with configuration files located outside the PGDATA directory

I tested it under Windows XP sp3.
All of configuration files(postgresql.conf pg_hba.conf pg_ident.conf) are in c:\data,
and data_dir is in C:\Program Files\PostgreSQL\9.0\data

Check the attchment, please.

Another question, after clone source with git I can not compile them:
Bad format filename 'src\bin\scripts\submake-libpq'
Former makefile like "createdb: createdb.o ... keywords.o"
Now it is "createdb: createdb.o ... keywords.o | submake-libpq"
How to do this?

------------------------------------------
SEARCHING JOB. I can work on C/C++.

Quan Zongliang <quanzongliang@gmail.com>

Attachments:

pg_ctl.difftext/plain; name=pg_ctl.diffDownload+74-35
#2Alvaro Herrera
alvherre@2ndquadrant.com
In reply to: Quan Zongliang (#1)
Re: Patch BUG #5103: "pg_ctl -w (re)start" fails with custom unix_socket_directory

Excerpts from Quan Zongliang's message of sáb nov 27 06:03:12 -0300 2010:

Hi, all

I created a pg_ctl patch to fix:
* BUG #5103: "pg_ctl -w (re)start" fails with custom unix_socket_directory
Allow pg_ctl to work properly with configuration files located outside the PGDATA directory

I think the way this should work is that you call postmaster with a new
switch and it prints out its configuration, after reading the
appropriate config file(s). That way it handles all the little details
such as figuring out the correct config file, hadle include files, etc.
This output would be presumably easier to parse and more trustworthy.

Right now we have --describe-config, which is missing the values for
each config option.

--
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

#3Quan Zongliang
quanzongliang@gmail.com
In reply to: Alvaro Herrera (#2)
Re: Patch BUG #5103: "pg_ctl -w (re)start" fails with custom unix_socket_directory

On Mon, 29 Nov 2010 10:29:17 -0300
Alvaro Herrera <alvherre@commandprompt.com> wrote:

Excerpts from Quan Zongliang's message of s锟斤拷b nov 27 06:03:12 -0300 2010:

Hi, all

I created a pg_ctl patch to fix:
* BUG #5103: "pg_ctl -w (re)start" fails with custom unix_socket_directory
Allow pg_ctl to work properly with configuration files located outside the PGDATA directory

I think the way this should work is that you call postmaster with a new
switch and it prints out its configuration, after reading the
appropriate config file(s). That way it handles all the little details
such as figuring out the correct config file, hadle include files, etc.
This output would be presumably easier to parse and more trustworthy.

Right now we have --describe-config, which is missing the values for
each config option.

Sorry for my late reply.

I will check the source of postmaster.

--
Quan Zongliang <quanzongliang@gmail.com>

#4Alvaro Herrera
alvherre@2ndquadrant.com
In reply to: Quan Zongliang (#3)
Re: Patch BUG #5103: "pg_ctl -w (re)start" fails with custom unix_socket_directory

Excerpts from Quan Zongliang's message of mar dic 21 18:36:11 -0300 2010:

On Mon, 29 Nov 2010 10:29:17 -0300
Alvaro Herrera <alvherre@commandprompt.com> wrote:

I think the way this should work is that you call postmaster with a new
switch and it prints out its configuration, after reading the
appropriate config file(s). That way it handles all the little details
such as figuring out the correct config file, hadle include files, etc.
This output would be presumably easier to parse and more trustworthy.

Sorry for my late reply.

I will check the source of postmaster.

Actually Bruce Momjian is now working on a different fix:
unix_socket_directory would be added to postmaster.pid, allowing pg_ctl
to find it.

--
Álvaro Herrera <alvherre@commandprompt.com>
The PostgreSQL Company - Command Prompt, Inc.
PostgreSQL Replication, Consulting, Custom Development, 24x7 support

#5Bruce Momjian
bruce@momjian.us
In reply to: Alvaro Herrera (#4)
Re: Patch BUG #5103: "pg_ctl -w (re)start" fails with custom unix_socket_directory

Alvaro Herrera wrote:

Excerpts from Quan Zongliang's message of mar dic 21 18:36:11 -0300 2010:

On Mon, 29 Nov 2010 10:29:17 -0300
Alvaro Herrera <alvherre@commandprompt.com> wrote:

I think the way this should work is that you call postmaster with a new
switch and it prints out its configuration, after reading the
appropriate config file(s). That way it handles all the little details
such as figuring out the correct config file, hadle include files, etc.
This output would be presumably easier to parse and more trustworthy.

Sorry for my late reply.

I will check the source of postmaster.

Actually Bruce Momjian is now working on a different fix:
unix_socket_directory would be added to postmaster.pid, allowing pg_ctl
to find it.

Yes, I will apply this patch tomorrow and it will be in 9.1. Thanks for
the report.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +

#6Quan Zongliang
quanzongliang@gmail.com
In reply to: Bruce Momjian (#5)
Re: Patch BUG #5103: "pg_ctl -w (re)start" fails with custom unix_socket_directory

On Wed, 22 Dec 2010 21:02:35 -0500 (EST)
Bruce Momjian <bruce@momjian.us> wrote:

Alvaro Herrera wrote:

Excerpts from Quan Zongliang's message of mar dic 21 18:36:11 -0300 2010:

On Mon, 29 Nov 2010 10:29:17 -0300
Alvaro Herrera <alvherre@commandprompt.com> wrote:

I think the way this should work is that you call postmaster with a new
switch and it prints out its configuration, after reading the
appropriate config file(s). That way it handles all the little details
such as figuring out the correct config file, hadle include files, etc.
This output would be presumably easier to parse and more trustworthy.

Sorry for my late reply.

I will check the source of postmaster.

Actually Bruce Momjian is now working on a different fix:
unix_socket_directory would be added to postmaster.pid, allowing pg_ctl
to find it.

Yes, I will apply this patch tomorrow and it will be in 9.1. Thanks for
the report.

Nice work.

--
Quan Zongliang <quanzongliang@gmail.com>