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

[Igor Sysoev <is@xxxxxxxxxxxxx>]

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 всего не сделаешь - путь 
> относительно префикса и имя файла не поменяешь.

Буду смотреть, но не понял, что ты имел в виду под не очень хорошей идей. 

> Да, и время из ошибок при старте пропало - что IMHO тоже не есть 
> хорошо.

Оно пропало из stderr и я думаю тут оно не особо нужно.
В error_log должно остаться.


-- 
Игорь Сысоев
http://sysoev.ru