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

Alexey V. Karagodov kav at karagodov.name
Fri Apr 24 10:54:49 MSD 2009


Hmm...  The next patch looks like a unified diff to me...
The text leading up to this was:
--------------------------
|Index: misc/GNUmakefile
|===================================================================
|--- misc/GNUmakefile	(revision 2048)
|+++ misc/GNUmakefile	(working copy)
--------------------------
File to patch:

On 24.04.2009, at 10:15, 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 всего не сделаешь - путь
>>> относительно префикса и имя файла не поменяешь.
>>
>> Буду смотреть, но не понял, что ты имел в виду под не очень хорошей  
>> идей.
>
> Новая версия, в которой исправлен перфикс для на старте:
> http://sysoev.ru/tmp/patch.0.7.52.1.prefix
>
>
> -- 
> Игорь Сысоев
> http://sysoev.ru
>






More information about the nginx-ru mailing list