Re: Postgres Pro build for windows

Hi Stefan!

Sorry, my feedback is also slow :)

Thanks for your comments.

The new patch is attached, some notes below.

09.01.2018 23:35, Stefan Kaltenbrunner пишет:
> On 12/22/2017 11:16 AM, Ivan E. Panchenko wrote:
>> Hi,
> Hi Ivan!
>
> Sorry for the slow feedback but see below.
>
>> 13.12.2017 13:09, Daniel Gustafsson пишет:
>>>> On 13 Dec 2017, at 11:05, Ivan E. Panchenko
>>>> <i(dot)panchenko(at)postgrespro(dot)ru> wrote:
>>>>
>>>> 13.12.2017 12:26, Daniel Gustafsson пишет:
>>>>>> On 13 Dec 2017, at 10:12, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> I have no objections, however I'm also extremely short of time
>>>>>> right now, as I'm only in the office for a couple more days before
>>>>>> Christmas and already have a huge backlog. If someone else can pick
>>>>>> this up it would be a big help.
>>>>> The markup in the patch looks fine, there are however two small
>>>>> typos that
>>>>> should be fixed before pushing:
>>>>>
>>>>> +including its old versions, provides automatic server
>>>>> preconfiguraion based
>>>>>
>>>>> s/preconfiguraion/preconfiguration/
>>>>>
>>>>> +sources are avaiable at github.
>>>>>
>>>>> s/avaiable/available/
>>>>>
>>>>> Further, since GitHub is a name I think we should capitalize it as
>>>>> they do (ie
>>>>> GitHub).
>>>> Daniel, thanks for your corrections. Sorry for the misprints.
>>>> The improved patch is attached.
>>> No worries. Updated patch looks good to me.
>> Can anybody install the patch ?
> I took a look at the patch wrt to applying it but I think there are
> still some improvements/clarifications to make:
>
> +<h2>Graphical installer by Postgres Pro</h2>
>
> From looking at "About us" on www.postgrespro.ru it seems that the
> company name is "Postgres Professional" and "Postgres Pro" is used for
> the actual (custom) software product - so maybe the above should be
> "Graphical installer by Postgres Professional"?
Ok, fixed!
>
> +
> +<p>
> +<a href="https://postgrespro.com/windows">PostgreSQL GUI installer</a>
> +by Postgres Professional supports 32 and 64 bit architectures for Windows,
>
> We use s/bit/Bit elsewhere on this page als "supports ... architectures
> for Windows" reads weird - Microsoft usually talks about "platforms" in
> that context
Also fixed
>
> +including its old versions, provides automatic server preconfiguration
> based
>
> not sure what "its old versions" is referring to?
Since Windows XP. Mentioned this in the text.
>
> +on available RAM size, corrects several issues with non-latin messages
> encoding,
>
> "corrects several issues with non-latin messages"? is that referring to
> some patch applied to postgresql sources? if yes shouldn't that our be
> in the main tree if it is an actual bug?
It's not a Postgres bug and not a Postgres patch, just setting a correct
codepage before running psql.
Clarified this in the text.

Regards,

Ivan
>
>
>
> Stefan
>