Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Burton, Ross
On 1 February 2013 11:37, Radu Moisan wrote: > By default the polkit will autodetect if systemd is available > and setup the configuraton accordingly, potentially > contaminating sstate. > In this patch systemd is explicitly enabled/disabled. > When enabled, polkit will use systemd for session tar

Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Radu Moisan
On 02/01/2013 12:27 PM, Burton, Ross wrote: On 1 February 2013 10:22, Radu Moisan wrote: I did change that manually because I assumed it was a typo. However it make sense to add consolekit as a runtime dependency, but the documentation specifically says PACKAGECONFIG will add to DEPENDS. How i

[OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Radu Moisan
By default the polkit will autodetect if systemd is available and setup the configuraton accordingly, potentially contaminating sstate. In this patch systemd is explicitly enabled/disabled. When enabled, polkit will use systemd for session tarcking, otherwise it will use ConsoleKit. Signed-off-by:

Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Burton, Ross
On 1 February 2013 10:22, Radu Moisan wrote: > I did change that manually because I assumed it was a typo. However it make > sense to add consolekit as a runtime dependency, but the documentation > specifically says PACKAGECONFIG will add to DEPENDS. How is it different in > this patch? The PACKAG

Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Martin Jansa
On Fri, Feb 01, 2013 at 12:22:01PM +0200, Radu Moisan wrote: > > On 02/01/2013 11:44 AM, Burton, Ross wrote: > > On 1 February 2013 09:22, Radu Moisan wrote: > >> enable/disable support as relevant > > Please write a more verbose comment, something along the lines of > > "systemd is auto-detected

Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Radu Moisan
On 02/01/2013 11:44 AM, Burton, Ross wrote: On 1 February 2013 09:22, Radu Moisan wrote: enable/disable support as relevant Please write a more verbose comment, something along the lines of "systemd is auto-detected unless disabled explicitly, potentially contaminating sstate". +# there is

Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Radu Moisan
On 02/01/2013 11:47 AM, Burton, Ross wrote: On 1 February 2013 09:39, Andreas Müller wrote: On Fri, Feb 1, 2013 at 10:22 AM, Radu Moisan wrote: enable/disable support as relevant Signed-off-by: Radu Moisan Signed-off-by: Andreas Müller What caused my sign-off - or is it a hint that meta-

Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Burton, Ross
On 1 February 2013 09:39, Andreas Müller wrote: > On Fri, Feb 1, 2013 at 10:22 AM, Radu Moisan wrote: >> enable/disable support as relevant >> >> Signed-off-by: Radu Moisan >> Signed-off-by: Andreas Müller > What caused my sign-off - or is it a hint that meta-oe/meta-systemd > need alignment wi

Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Radu Moisan
On 02/01/2013 11:39 AM, Andreas Müller wrote: On Fri, Feb 1, 2013 at 10:22 AM, Radu Moisan wrote: enable/disable support as relevant Signed-off-by: Radu Moisan Signed-off-by: Andreas Müller What caused my sign-off - or is it a hint that meta-oe/meta-systemd need alignment with this? You

Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Burton, Ross
On 1 February 2013 09:22, Radu Moisan wrote: > enable/disable support as relevant Please write a more verbose comment, something along the lines of "systemd is auto-detected unless disabled explicitly, potentially contaminating sstate". > +# there is no --enable/--disable option for consolekit a

Re: [OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Andreas Müller
On Fri, Feb 1, 2013 at 10:22 AM, Radu Moisan wrote: > enable/disable support as relevant > > Signed-off-by: Radu Moisan > Signed-off-by: Andreas Müller What caused my sign-off - or is it a hint that meta-oe/meta-systemd need alignment with this? Andreas

[OE-core] [PATCH] polkit: Enable systemd support

2013-02-01 Thread Radu Moisan
enable/disable support as relevant Signed-off-by: Radu Moisan Signed-off-by: Andreas Müller --- meta/recipes-extended/polkit/polkit_0.104.bb |7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/meta/recipes-extended/polkit/polkit_0.104.bb b/meta/recipes-extended/polkit/