All files from this thread

File Name	File Size		Posted by	Date
patch-nginx-installconf.txt	481 bytes	open | download	Maxim Dounin	06/26/2009	Read message

Hello!

On Fri, Jun 26, 2009 at 05:27:10PM +0100, Pedro Mata-Mouros Fonseca wrote:

> Yes, I'm compiling from sources. I found out what the problem is:
>
> $ ./configure --prefix=/servers/nginx-0.7.60 --conf-path=/etc/ngix.conf
> $ make
> # make install
> [SNIP]
> test -f '/etc/ngix.conf' || cp conf/nginx.conf '/etc'
> cp conf/nginx.conf '/etc/nginx.conf.default'
> test -d '/servers/nginx-0.7.60/logs' || mkdir -p '/servers/
> nginx-0.7.60/logs'
> test -d '/servers/nginx-0.7.60/logs' || mkdir -p '/
> servers/nginx-0.7.60/logs'
> test -d '/servers/nginx-0.7.60/html' || cp -r html '/
> servers/nginx-0.7.60'
> test -d '/servers/nginx-0.7.60/logs' || mkdir -p '/
> servers/nginx-0.7.60/logs'
> make[1]: Leaving directory `/servers/sources/nginx-0.7.60'
>
> In the configure line I made a mistake and stated /etc/ngix.conf as the
> config file (notice the lack of the n). When installing, the script stats
> /etc/ngix.conf, and as it doesn't exist, it copies the default
> nginx.conf, not changing its name. Since I use a init.d script for
> starting up nginx, configured to use /etc/nginx.conf, this file always is
> replaced in this situation on every new install.
>
> It's not really important, maybe it can be fixed when somebody has 1
> minute spare.
> Thanks.

Thank you, nice catch. It's regression introduced in 0.6.6.
Patch attached.

Maxim Dounin

>
>
> On Jun 26, 2009, at 4:11 PM, Jim Ohlstein wrote:
>
>> That has never happened on my servers. I use a different path but that
>> should not matter.
>>
>> Are you compiling yourself?
>>
>> Jim
>>
>> ------Original Message------
>> From: Pedro Mata-Mouros Fonseca
>> Sender: owner-nginx@sysoev.ru
>> To: nginx@sysoev.ru
>> ReplyTo: nginx@sysoev.ru
>> Subject: nginx.conf rewritten
>> Sent: Jun 26, 2009 11:02 AM
>>
>> Greetings,
>>
>> It seems that /etc/nginx.conf gets replaced by the default in each new
>> nginx install. Do you confirm this? Any chance it doesn't replace that
>> file, if it exists already, by default?
>>
>> Thank you.
>>
>> Pedro.
>>
>>
>>
>> Sent from my Verizon Wireless BlackBerry
>>
>>
>
>