Re: Postgres Pro build for windows

From: "Ivan E(dot) Panchenko" <i(dot)panchenko(at)postgrespro(dot)ru>
To: pgsql-www(at)lists(dot)postgresql(dot)org, Stefan Kaltenbrunner <stefan(at)kaltenbrunner(dot)cc>
Subject: Re: Postgres Pro build for windows
Date: 2018-03-23 13:53:36
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-www

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 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="">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.


> Stefan

Attachment Content-Type Size
pgpro-windows.patch text/x-patch 1.1 KB

In response to


Browse pgsql-www by date

  From Date Subject
Next Message Joe Krill 2018-03-27 17:13:59 Wiki editor request: Fixing Sequences
Previous Message Teodor Sigaev 2018-03-23 11:27:14 Re: [HACKERS] GSoC 2017: weekly progress reports (week 6)