Re: Postgres Pro build for windows
От | Ivan E. Panchenko |
---|---|
Тема | Re: Postgres Pro build for windows |
Дата | |
Msg-id | cc86f65b-5f3a-14b1-62ac-ad863bcefc5d@postgrespro.ru обсуждение исходный текст |
Ответ на | Re: Postgres Pro build for windows (Stefan Kaltenbrunner <stefan@kaltenbrunner.cc>) |
Ответы |
Re: Postgres Pro build for windows
("Ivan E. Panchenko" <i.panchenko@postgrespro.ru>)
|
Список | 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.panchenko@postgrespro.ru> wrote: >>>> >>>> 13.12.2017 12:26, Daniel Gustafsson пишет: >>>>>> On 13 Dec 2017, at 10:12, Dave Page <dpage@pgadmin.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 >
Вложения
В списке pgsql-www по дате отправления: