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

[Igor Sysoev <is@xxxxxxxxxxxxx>]

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 пишет - мало 
> не покажется. :)

Это при каких настройках ?

> А вот без явного указания конфига не запускается:
> 
> $ 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