Re: большой патч в пре дверии 0.7.53

[Maxim Dounin <mdounin@xxxxxxxxxx>]

Hello!

On Fri, Apr 24, 2009 at 01:29:12PM +0400, Igor Sysoev wrote:

> On Fri, Apr 24, 2009 at 12:09:41PM +0400, Maxim Dounin wrote:
> 
> > Hello!
> > 
> > On Fri, Apr 24, 2009 at 10:15:30AM +0400, Igor Sysoev wrote:
> > 
> > > On Fri, Apr 24, 2009 at 08:30:18AM +0400, Igor Sysoev wrote:
> > > 
> > > > On Fri, Apr 24, 2009 at 01:27:08AM +0400, Maxim Dounin wrote:
> > > > 
> > > > > Hello!
> > > > > 
> > > > > On Thu, Apr 23, 2009 at 11:14:59PM +0400, Igor Sysoev wrote:
> > > > > 
> > > > > > Будущее уже сегодня:
> > > > > > http://sysoev.ru/tmp/patch.0.7.52.prefix
> > > > > > 
> > > > > > В патче много изменений, связанных с ключём -p prefix и выводом 
> > > > > > ошибок
> > > > > > на старте и я хочу протестировать его до релиза.
> > > > > 
> > > > > Патч чисто не накладывается, ибо у простых смертных нет файла 
> > > > > auto/configure.  При указании руками что патчить надо configure - 
> > > > > наложился чисто.
> > > > > 
> > > > > >     *) Изменение: теперь лог, указанный в --error-log-path, 
> > > > > > создаётся с 
> > > > > >        самого начала работы.
> > > > > 
> > > > > $ ./configure 
> > > > > --prefix="/usr/home/mdounin/work/nginx-generic/debug-nginx-install" 
> > > > > ...
> > > > > 
> > > > > $ make
> > > > > ...
> > > > > 
> > > > > $ objs/nginx -h
> > > > > nginx version: nginx/0.7.52
> > > > > ...
> > > > >   -p prefix     : set prefix path (default: 
> > > > > /usr/home/mdounin/work/nginx-generic/debug-nginx-install/)
> > > > > ...
> > > > > 
> > > > > $ ktrace objs/nginx
> > > > > [emerg]: could not open error log file: open() "logs/error.log" 
> > > > > failed (2: No such file or directory)
> > > > > 
> > > > > $ kdump
> > > > > ...
> > > > >  36042 nginx    NAMI  "logs/error.log"
> > > > >  36042 nginx    RET   open -1 errno 2 No such file or directory
> > > > > ...
> > > > > 
> > > > > Но даже если оно будет работать - IMHO это не очень хорошая идея.  
> > > > > Как минимум test suite оно мне сломает, ибо будет пытаться писать 
> > > > > в дефоултные места - куда у него прав обычно нет.  Ну и вообще 
> > > > > имеется шанс сломать много ранее работавших конфигураций.
> > > > > 
> > > > > При этом насколько я понимаю никакими средствами кроме -p это не 
> > > > > лечится, даже -g не поможет.  А через -p всего не сделаешь - путь 
> > > > > относительно префикса и имя файла не поменяешь.
> > > > 
> > > > Буду смотреть, но не понял, что ты имел в виду под не очень хорошей 
> > > > идей. 
> > 
> > Я имел ввиду что вне зависимости от опций сборки должна быть 
> > возможность запустить nginx без каких-либо попыток что-то писать в 
> > какие-либо файлы, пути которых прилетели из сборки.
> > 
> > Сейчас я это делаю через -c и конфиг в котором явно указываются 
> > error_log и pid_file (ну и для http всякого).
> > 
> > Как я это буду делать после вышеозначенного изменения - не 
> > представляю.
> 
> Открытие error_log с самого старта нужно, как минимум, для логирования
> ошибок запуска при загрузке сервера.
> 
> А для отладки я всегда использовал
>     --prefix=$(PWD)
>     --error-log-path=stderr
> 
> Для тестов сейчас можно делать так:
>     --prefix=.
>     --error-log-path=stderr

Я понимаю что собрать можно так как хочется.  Вопрос что делать 
если оно *уже* собрано неизвестно как, и надо запустить тест и/или 
другую копию - не трогая при этом системные пути вообще.

Реньше это можно было сделать, после патча - нельзя.

> > > Новая версия, в которой исправлен перфикс для на старте:
> > > http://sysoev.ru/tmp/patch.0.7.52.1.prefix
> > 
> > $ make test
> > ...
> > 2009/04/24 11:48:09 [notice] 42427#0: exit
> > 2009/04/24 11:48:09 [debug] 42427#0: run cleanup: 080EBCB4
> > 2009/04/24 11:48:09 [debug] 42427#0: cleanup resolver
> > 2009/04/24 11:48:09 [debug] 42427#0: free: 080DB000, unused: 0
> > 2009/04/24 11:48:09 [debug] 42427#0: free: 080E4000, unused: 0
> > 2009/04/24 11:48:09 [debug] 42427#0: free: 080EB000, unused: 3767
> > ngx_http_upstream_keepalive/t/memcached-keepalive....ok
> > All tests successful.
> > Files=22, Tests=193, 52 wallclock secs ( 0.73 usr  3.34 sys +  
> > 2.30 cusr  8.95 csys = 15.32 CPU)
> > Result: PASS
> > 
> > В том смысле что работать оно работает, но в stderr пишет - мало 
> > не покажется. :)
> 
> Это при каких настройках ?

$ nginx/objs/nginx -V
nginx version: nginx/0.7.52
built by gcc 3.4.6 [FreeBSD] 20060305
configure arguments: --with-debug --with-cc-opt=-D NGX_DEBUG_MALLOC 
--with-http_flv_module --with-mail --with-mail_ssl_module --with-ipv6 
--add-module=/usr/home/mdounin/work/nginx-generic/ngx_http_upstream_keepalive 
--add-module=/usr/home/mdounin/work/nginx-generic/ngx_http_bytes_filter_module 
--prefix=/usr/home/mdounin/work/nginx-generic/debug-nginx-install

$ cat /usr/home/mdounin/work/nginx-generic/t.conf
error_log /tmp/error.log debug;
events { }

$ nginx/objs/nginx -c /usr/home/mdounin/work/nginx-generic/t.conf
... (debug log тут)

$ ls -la /tmp/error.log
ls: /tmp/error.log: No such file or directory

Maxim Dounin

> 
> > А вот без явного указания конфига не запускается:
> > 
> > $ ktrace nginx/objs/nginx
> > [emerg]: open() "conf/conf/nginx.conf" failed (2: No such file or directory)
> > 
> > $ kdump
> > ...
> >  42443 nginx    CALL  open(0x80db12b,0,0)
> >  42443 nginx    NAMI  "conf/conf/nginx.conf"
> >  42443 nginx    RET   open -1 errno 2 No such file or directory
> > ...
> 
> Буду смотреть.
> 
> 
> -- 
> Игорь Сысоев
> http://sysoev.ru
>