On Thu, Jan 7, 2016 at 4:53 AM, Alvaro Herrera <alvherre@2ndquadrant.com> wrote:
> Michael Paquier wrote:
>> Hi all,
>>
>> I just bumped into the issue defined in $subject.
>> Attached is a patch.
>
> I don't understand the parenthical remark. Do you mean that if you pass
> "--bindir" without an argument it uses PATH? That would be weird; but
> if that's how it works, then it should look like --bindir[=DIR].
>From pg_regress.c:
/* "--bindir=" means to use PATH */
if (strlen(optarg))
bindir = strdup(optarg);
else
bindir = NULL;
break;
> If you
> don't mean that, then I think the proposed wording is confusing. Maybe
> "PATH is searched if this option not given"?
That's trickier. PGBINDIR is used if this option is not given :)
Do you prefer the attached?
--
Michael