19 января 2017 г., 13:08 пользователь Ruslan Ermilov <ru@nginx.com> написал:

> Илья, приветствую!
>
> On Wed, Jan 18, 2017 at 09:58:36AM +0500, Илья Шипицин wrote:
> > 19 декабря 2016 г., 17:26 пользователь Ruslan Ermilov <ru@nginx.com>
> > написал:
> >
> > > Приветствую!
> > >
> > > On Sun, Dec 18, 2016 at 10:33:19PM +0500, Илья Шипицин wrote:
> > > > Руслан, добрый день!
> > > >
> > > > а может быть вы сможете закомитить ту версию патча, которую допустимо
> > > > закомитить ?
> > >
> > > В смысле Вы предлагаете мне закоммитить патч Максима, который явно
> > > обозначил (даже дважды), что не хотел бы его коммитить без тщательного
> > > тестирования? Боюсь, в таком случае я знаю, каким будет следующий
> > > коммит. :)
> > >
> > > Мне видится более правильным вариант, если Вы сообщите о результатах
> > > проведённого Вами уже видимо трёхлетнего тестирования патча, который
> > > предлагал Максим.
> > >
> >
> > Руслан, может быть такой вариант - дефолтное поведение оставить
> неизменным,
> > и добавить настройку, скажем "ядреная_оптимизация_кипэлайв on",
> энтузиасты
> > включат (на свой риск), протестируют, и можно будет думать о том, чтобы
> > поменять дефолтное поведение.
>
> Лично я в такой ручке вижу мало смысла. Есть готовый патч. Желающие
> могут его протестировать и сообщить о рез-тах тестирования. После
> чего патч будет скорее всего закоммичен.
>

кажется, тут может возникнуть двусмысленность.
тестировали мы мою версию патча.

вариант Максима я вижу менее логичным (ему хочется сравнивать версии
протокола http аналогичным образом, как это делается для версий библиотеки
openssl), функционально это одно и то же, но насчет тестирования патча
Максима предлагаю вопросы задавать ему.

и по прежнему, если вопрос в тестировании, предлагаю рассмотреть вариант с
"рубильником"


> _______________________________________________
> nginx-ru mailing list
> nginx-ru@nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-ru
>
_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru