Bug#701112: (no subject)

2013-11-24 Thread Michael Lustfield
In debian/nginx-common.preinst: # http://bugs.debian.org/701112 if [ `stat -c '%U:%G.%a' /var/log/nginx` == 'root:root.755' ]; then chown root:adm /var/log/nginx chmod 750 /var/log/nginx fi This has been changed. I will create a NEWS entry and push. It will be resolved in

Bug#701112: (no subject)

2013-03-12 Thread Steven Chamberlain
Hi, On 12/03/13 03:33, Michael Lustfield wrote: In debian/nginx-common.postinst we have: configure) logdir=/var/log/nginx # Ensure existance and right state of log files and directory if [ ! -d $logdir -a ! -L $logdir ]; then mkdir $logdir chown www-data:adm

Bug#701112: (no subject)

2013-03-12 Thread Michael Lustfield
precautionary - That would mean we assume that making the change won't break anything. We're setting this for new installs but forcing it on already deployed systems wouldn't be a good idea. We could add a NEWS entry to recommend making this change. It's definitely not a good idea to force it to

Bug#701112: (no subject)

2013-03-12 Thread Steven Chamberlain
On 12/03/13 13:57, Michael Lustfield wrote: precautionary - That would mean we assume that making the change won't break anything. We're setting this for new installs but forcing it on already deployed systems wouldn't be a good idea. We could add a NEWS entry to recommend making this change.

Bug#701112: (no subject)

2013-03-11 Thread Michael Lustfield
In debian/nginx-common.postinst we have: configure) logdir=/var/log/nginx # Ensure existance and right state of log files and directory if [ ! -d $logdir -a ! -L $logdir ]; then mkdir $logdir chown www-data:adm $logdir chmod 0750 $logdir fi This should create