qboosh: SPECS openldap.spec SOURCES openldap-pidfile.patch

[Jakub Bogusz]

On Sun, Aug 12, 2001 at 09:34:19PM +0200, Tomasz Kłoczko wrote:
> On Sun, 12 Aug 2001, PLD CVS wrote:
> 
> > Module name:	SOURCES
> > Changes by:	qboosh	01/08/12 21:25:57
> > 
> > Modified files:
> > 	.              : openldap.spec 
> > Removed files:
> > 	.              : openldap-pidfile.patch 
> > 
> > Log message:
> > - removed pidfile patch (from rawhide) - it's buggy.
> >   It saves pid before fork - pid of process which won't exit after slapd start.
> >   Note that slapd itself has an option to save pid to file - but it doesn't
> >   work when slapd is run not as root (no +w perm in /var/run).
> >   For now - don't save pid file, let rc-scripts use pidof. Does someone have
> >   better idea?
> 
> Ostatecznie możnaby zrobić tak, że dać prawa zapisu dla mail w np. 
> /var/run/openldap i tam zakładać plik z PIDem (?).

Dla slapd raczej.

> Inne rozwiazanie to wtrącenie zrzucania pidfile po fork() a przed zmianą 
> uid (nie przeglądałem źreódęł i nie wiem jak to tam dokąłdnie jest 
> zrobione).

Ja zaglądałem - zrzuca uprawnienia dużo wcześniej.

O podkatalogu w /var/run myślałem, ale nie mamy mechanizmu do odczytu pidów
z innego miejsca niż /var/run/`basename $demon`.pid .
Może warto coś takiego dodać, np jako dodatkowy parametr do funkcji
pidofproc i killproc?


-- 
Jakub Bogusz
http://prioris.mini.pw.edu.pl/~qboosh/