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

[Maxim Dounin <mdounin@xxxxxxxxxx>]

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 тоже не есть 
хорошо.

Maxim Dounin

>     *) Добавление: теперь ошибки и предупреждения при старте записываются в 
>        error_log и выводятся на stderr.
> 
>     *) Добавление: при сборке с параметром --prefix=. nginx использует 
>        текущий каталог в качестве префикса.
> 
>     *) Добавление: ключ -p.
> 
>     *) Добавление: ключ -s на Unix-платформах.
> 
>     *) Добавление: ключи -? и -h.
> 
>     *) Добавление: теперь ключи можно задавать в сжатой форме.
> 
>     *) Исправление: директива fastcgi_cache_min_uses не работала.
> 
> 
> -- 
> Игорь Сысоев
> http://sysoev.ru
>