From: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
---|---|
To: | Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, Asif Naeem <anaeem(dot)it(at)gmail(dot)com> |
Cc: | PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Fix broken Install.bat when target directory contains a space |
Date: | 2015-07-03 18:57:00 |
Message-ID: | 5596DAFC.3020101@iki.fi |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 04/21/2015 04:02 PM, Michael Paquier wrote:
> On Tue, Apr 21, 2015 at 4:33 PM, Asif Naeem <anaeem(dot)it(at)gmail(dot)com> wrote:
>
>> The v2 patch looks good to me, just a minor concern on usage message i.e.
>>
>> C:\PG\postgresql\src\tools\msvc>install
>>> Invalid command line options.
>>> Usage: "install.bat <targetdir> [installtype]"
>>> installtype: client
>>
>>
>> It seems that there are two install options i.e. client, all (any other
>> string other than client is being considered or treated as all), the
>> following install command works i.e.
>>
>> install C:\PG\postgresql\inst option_does_not_exist
>>
>>
>> As your patch effects this area of code, I thought to share these findings
>> with you,o BTW, it is a minor thing that can be handled in another patch,
>
> Well, that's the same behavior that this script has been having for ages.
> Let's just update the usage message to mention both "all" and "client". I
> see no point in breaking a behavior that has been like that for ages, and
> the main point of this patch is to fix the install path issue.
Hmm. Why is install.bat not like build.bat, i.e. just a thin wrapper
that just calls install.pl, passing all arguments?
- Heikki
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2015-07-03 19:50:43 | Re: psql :: support for \ev viewname and \sv viewname |
Previous Message | Petr Korobeinikov | 2015-07-03 18:24:19 | Re: psql :: support for \ev viewname and \sv viewname |