Re: typos in comments and user docs
От | Justin Pryzby |
---|---|
Тема | Re: typos in comments and user docs |
Дата | |
Msg-id | 20200207041706.GO403@telsasoft.com обсуждение исходный текст |
Ответ на | Re: typos in comments and user docs (Amit Kapila <amit.kapila16@gmail.com>) |
Ответы |
Re: typos in comments and user docs
|
Список | pgsql-hackers |
On Fri, Feb 07, 2020 at 09:26:04AM +0530, Amit Kapila wrote: > On Fri, Feb 7, 2020 at 8:41 AM Justin Pryzby <pryzby@telsasoft.com> wrote: > > > > On Fri, Feb 07, 2020 at 08:33:40AM +0530, Amit Kapila wrote: > > > On Thu, Feb 6, 2020 at 7:26 PM Justin Pryzby <pryzby@telsasoft.com> wrote: > > > > > > > > On Thu, Feb 06, 2020 at 04:43:18PM +0530, Amit Kapila wrote: > > > > > On Thu, Feb 6, 2020 at 10:45 AM Michael Paquier <michael@paquier.xyz> wrote: > > > > > > > > > > > > On Thu, Feb 06, 2020 at 08:47:14AM +0530, Amit Kapila wrote: > > > > > > > Your changes look fine to me on the first read. I will push this to > > > > > > > HEAD unless there are any objections. If we want them in > > > > > > > back-branches, we might want to probably segregate the changes based > > > > > > > on the branch until those apply. > > > > > > > > > > > > +1. It would be nice to back-patch the user-visible changes in the > > > > > > docs. > > > > > > > > > > > > > > > > Fair enough, Justin, is it possible for you to segregate the changes > > > > > that can be backpatched? > > > > > > > > Looks like the whole patch can be applied to master and v12 [0]. > > > > > I tried your patch master and it failed to apply. > (Stripping trailing CRs from patch; use --binary to disable.) > patching file doc/src/sgml/bloom.sgml > (Stripping trailing CRs from patch; use --binary to disable.) > patching file doc/src/sgml/config.sgml > Hunk #1 FAILED at 4318. > 1 out of 1 hunk FAILED -- saving rejects to file doc/src/sgml/config.sgml.rej I think you applied the first patch, which I corrected here. https://www.postgresql.org/message-id/20200206135640.GG403%40telsasoft.com Just rechecked it works for master and v12. $ git checkout -b test2 origin/master Branch test2 set up to track remote branch master from origin. Switched to a new branch 'test2' $ patch -p1 <0001-spelling-and-typos.patch patching file doc/src/sgml/bloom.sgml patching file doc/src/sgml/ref/alter_table.sgml patching file doc/src/sgml/sources.sgml patching file src/backend/access/transam/README.parallel patching file src/backend/storage/buffer/bufmgr.c patching file src/backend/storage/sync/sync.c patching file src/include/access/tableam.h $ patch -p1 <0001-spelling-and-typos.patch patching file doc/src/sgml/bloom.sgml patching file doc/src/sgml/ref/alter_table.sgml Hunk #1 succeeded at 220 (offset -2 lines). patching file doc/src/sgml/sources.sgml patching file src/backend/access/transam/README.parallel patching file src/backend/storage/buffer/bufmgr.c Hunk #1 succeeded at 4268 (offset -23 lines). patching file src/backend/storage/sync/sync.c patching file src/include/access/tableam.h Hunk #1 succeeded at 1167 (offset -18 lines). The bloom patch there works for v11. Attached now another version for v10-.
Вложения
В списке pgsql-hackers по дате отправления: