[Box Backup-dev] Re: [Box Backup-commit] COMMIT r2191 - box/trunk/contrib/debian

[Chris Wilson]

Hi James,

On Sun, 15 Jun 2008, James O'Gorman wrote:

> >  BBACKUPD=@sbindir_expanded@/bbackupd
> >  CONFIG=@sysconfdir_expanded@/box/bbackupd.conf
> > -PIDFILE=@localstatedir_expanded@/bbackupd.pid
> > +PIDFILE=@localstatedir_expanded@/bbackupd/bbackupd.pid
> 
> Should that not be run/bbackupd.pid (as per bbstored below)?

Well spotted, but the default for bbackupd-config, as far as I can tell, 
is to place all run-state files, including the PID file, in /var/bbackupd.

If users choose a different directory for running state than the 
recommended one, or change the PidFile in bbackupd.conf, then they will 
have to hack their initscripts to match.

I thought about changing the default location of the PID file to be 
@localstatedir_expanded@/run, but that would break backwards 
compatibility. I'm not averse to doing that for 0.11 release if people 
want it, but I didn't want to just go ahead and do it without feedback 
from people.

So, people, if you'd prefer your bbackupd PID file in /var/run or 
/var/bbackupd, please let me know :-)

Cheers, Chris.
-- 
_____ __     _
\  __/ / ,__(_)_  | Chris Wilson <0000 at qwirx.com> - Cambs UK |
/ (_/ ,\/ _/ /_ \ | Security/C/C++/Java/Ruby/Perl/SQL Developer |
\ _/_/_/_//_/___/ | We are GNU : free your mind & your software |