Re: ldapd: add -r option to specify datadir path

2016-02-02 Thread Mark Kettenis
> From: j...@wxcvbn.org (=?utf-8?Q?J=C3=A9r=C3=A9mie_Courr=C3=A8ges-Anglas?=) > Date: Tue, 02 Feb 2016 19:23:25 +0100 > > j...@wxcvbn.org (Jérémie Courrèges-Anglas) writes: > > > Sebastien Marie writes: > > > >> On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: > >>> Hi, > >>> >

Re: ldapd: add -r option to specify datadir path

2016-02-02 Thread Jérémie Courrèges-Anglas
j...@wxcvbn.org (Jérémie Courrèges-Anglas) writes: > Sebastien Marie writes: > >> On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: >>> Hi, >>> >>> i'm tinkering with ldapd and writing regress tests for it, and to >>> allow running independent instances (with separate port/control >

Re: ldapd: add -r option to specify datadir path

2016-02-02 Thread Gleydson Soares
> Here's a similar diff for ldapctl, thoughts? With your tweaks sounds much better. OK gsoares@

Re: ldapd: add -r option to specify datadir path

2016-02-02 Thread Landry Breuil
On Tue, Feb 02, 2016 at 01:58:18PM +0100, Jérémie Courrèges-Anglas wrote: > Gleydson Soares writes: > > >> Thinking about it .. would a call to access(2) with R_OK|W_OK|R_OK|F_OK > >> satisfy > >> everyone ? Or only F_OK ? > > > > Sounds better than chdir(2), but it will lack if datadir was pass

Re: ldapd: add -r option to specify datadir path

2016-02-02 Thread Jérémie Courrèges-Anglas
Gleydson Soares writes: >> Thinking about it .. would a call to access(2) with R_OK|W_OK|R_OK|F_OK >> satisfy >> everyone ? Or only F_OK ? > > Sounds better than chdir(2), but it will lack if datadir was passed to > access(2) > without including trailing "/" > > eg with access(datadir, F_OK): >

Re: ldapd: add -r option to specify datadir path

2016-02-02 Thread Gleydson Soares
> Thinking about it .. would a call to access(2) with R_OK|W_OK|R_OK|F_OK > satisfy > everyone ? Or only F_OK ? Sounds better than chdir(2), but it will lack if datadir was passed to access(2) without including trailing "/" eg with access(datadir, F_OK): $ touch /home/gsoares/testfile <- creat

Re: ldapd: add -r option to specify datadir path

2016-02-01 Thread Gleydson Soares
On Mon, Feb 1, 2016 at 5:13 PM, Jérémie Courrèges-Anglas wrote: > Gleydson Soares writes: > >> Hi Landry, >> >> On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: >>> Hi, >>> >>> i'm tinkering with ldapd and writing regress tests for it, and to >>> allow running independent instances

Re: ldapd: add -r option to specify datadir path

2016-02-01 Thread Landry Breuil
On Mon, Feb 01, 2016 at 09:13:00PM +0100, Jérémie Courrèges-Anglas wrote: > Gleydson Soares writes: > > > Hi Landry, > > > > On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: > >> Hi, > >> > >> i'm tinkering with ldapd and writing regress tests for it, and to > >> allow running inde

Re: ldapd: add -r option to specify datadir path

2016-02-01 Thread Jérémie Courrèges-Anglas
Gleydson Soares writes: > Hi Landry, > > On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: >> Hi, >> >> i'm tinkering with ldapd and writing regress tests for it, and to >> allow running independent instances (with separate port/control >> socket/etc) i needed to add the possibility

Re: ldapd: add -r option to specify datadir path

2016-02-01 Thread Landry Breuil
On Mon, Feb 01, 2016 at 07:37:34PM +0100, Jérémie Courrèges-Anglas wrote: > j...@wxcvbn.org (Jérémie Courrèges-Anglas) writes: > > > Sebastien Marie writes: > > > >> On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: > >>> Hi, > >>> > >>> i'm tinkering with ldapd and writing regress

Re: ldapd: add -r option to specify datadir path

2016-02-01 Thread Gleydson Soares
Hi Landry, On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: > Hi, > > i'm tinkering with ldapd and writing regress tests for it, and to > allow running independent instances (with separate port/control > socket/etc) i needed to add the possibility to specify an alternative > datadir

Re: ldapd: add -r option to specify datadir path

2016-02-01 Thread Jérémie Courrèges-Anglas
j...@wxcvbn.org (Jérémie Courrèges-Anglas) writes: > Sebastien Marie writes: > >> On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: >>> Hi, >>> >>> i'm tinkering with ldapd and writing regress tests for it, and to >>> allow running independent instances (with separate port/control >

Re: ldapd: add -r option to specify datadir path

2016-02-01 Thread Jérémie Courrèges-Anglas
Sebastien Marie writes: > On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: >> Hi, >> >> i'm tinkering with ldapd and writing regress tests for it, and to >> allow running independent instances (with separate port/control >> socket/etc) i needed to add the possibility to specify an

Re: ldapd: add -r option to specify datadir path

2016-02-01 Thread Sebastien Marie
On Sun, Jan 31, 2016 at 09:39:52AM +0100, Landry Breuil wrote: > Hi, > > i'm tinkering with ldapd and writing regress tests for it, and to > allow running independent instances (with separate port/control > socket/etc) i needed to add the possibility to specify an alternative > datadir, which was

ldapd: add -r option to specify datadir path

2016-01-31 Thread Landry Breuil
Hi, i'm tinkering with ldapd and writing regress tests for it, and to allow running independent instances (with separate port/control socket/etc) i needed to add the possibility to specify an alternative datadir, which was so far #defined in the code. Patch is pretty simple and works fine, i'm ope