Re: fixing pg_ctl with relative paths - Mailing list pgsql-hackers

From Fujii Masao
Subject Re: fixing pg_ctl with relative paths
Date
Msg-id CAHGQGwFT+qP+Dkk00SBvOEvtw0RBy=tF0jLmFEZtGFzM2RVYOw@mail.gmail.com
Whole thread Raw
In response to fixing pg_ctl with relative paths  (Josh Kupershmidt <schmiddy@gmail.com>)
Responses Re: fixing pg_ctl with relative paths
List pgsql-hackers
On Wed, Jun 26, 2013 at 2:36 PM, Hari Babu <haribabu.kommi@huawei.com> wrote:
> On June 26, 2013 5:02 AM Josh Kupershmidt wrote:
>>Thanks for the feedback. Attached is a rebased version of the patch with
> the two small issues noted fixed.

The following description in the document of pg_ctl needs to be modified?
   restart might fail if relative paths specified were specified on
the command-line during server start.

+#define DATADIR_SPEC    "\"-D\" \""
+
+    datadir = strstr(post_opts, DATADIR_SPEC);

Though this is a corner case, the patch doesn't seem to handle properly the case
where "-D" appears as other option value, e.g., -k option value, in
postmaster.opts
file.

Just idea to work around that problem is to just append the specified -D option
and value to post_opts. IOW, -D option and value appear twice in post_opts.
In this case, posteriorly-located ones are used in the end. Thought?

Regards,

-- 
Fujii Masao



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Add more regression tests for dbcommands
Next
From: Fujii Masao
Date:
Subject: Re: Review: Patch to compute Max LSN of Data Pages