Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-29 Thread Tomasz Torcz
On Tue, Mar 26, 2013 at 04:07:47PM -0300, Cristian Rodríguez wrote: > El 26/03/13 15:17, Bill Nottingham escribió: > >Lukas Nykryn (lnyk...@redhat.com) said: > >>--- > >> TODO | 2 -- > >> man/systemd.unit.xml.in | 8 > >> src/core/conditio

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-29 Thread Oleksii Shevchuk
> Well, we can discuss extending ExecStartPre= ExecStartPre= is not about that anyway, because it can be used only in .service units. > but I am pretty sure we shouldn't involve the process forking state > machine in the condition logic. I don't like it too. But can we have other alternatives?

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-29 Thread Lennart Poettering
On Fri, 29.03.13 18:00, Oleksii Shevchuk (shevc...@iit.kharkov.ua) wrote: > > > Ideas? > > Just provide ConditionExec=. People should build their own checkers. You > still can implement several as stand-alone executables maybe.. Well, we can discuss extending ExecStartPre= so that you can confi

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-29 Thread Cristian Rodríguez
On Fri 29 Mar 2013 01:00:59 PM CLST, Oleksii Shevchuk wrote: Ideas? Just provide ConditionExec=. People should build their own checkers. You still can implement several as stand-alone executables maybe.. I dont think so, If there are conditions implemented there should be implemented in syste

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-29 Thread Oleksii Shevchuk
> Ideas? Just provide ConditionExec=. People should build their own checkers. You still can implement several as stand-alone executables maybe.. ___ systemd-devel mailing list systemd-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listi

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-29 Thread Lennart Poettering
On Tue, 26.03.13 14:17, Bill Nottingham (nott...@redhat.com) wrote: > > Lukas Nykryn (lnyk...@redhat.com) said: > > --- > > TODO | 2 -- > > man/systemd.unit.xml.in | 8 > > src/core/condition.c | 16 > >

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-27 Thread Kok, Auke-jan H
On Wed, Mar 27, 2013 at 12:22 AM, Tollef Fog Heen wrote: > ]] Cristian Rodríguez > >> El 26/03/13 15:17, Bill Nottingham escribió: >> > Lukas Nykryn (lnyk...@redhat.com) said: >> >> --- >> >> TODO | 2 -- >> >> man/systemd.unit.xml.in | 8 +++

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-27 Thread Lukáš Nykrýn
St 27. březen 2013, 13:21:28 CET, Zbigniew Jędrzejewski-Szmek napsal: On Wed, Mar 27, 2013 at 08:22:14AM +0100, Tollef Fog Heen wrote: ]] Cristian Rodríguez El 26/03/13 15:17, Bill Nottingham escribió: Lukas Nykryn (lnyk...@redhat.com) said: --- TODO | 2

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-27 Thread Zbigniew Jędrzejewski-Szmek
On Wed, Mar 27, 2013 at 08:22:14AM +0100, Tollef Fog Heen wrote: > ]] Cristian Rodríguez > > > El 26/03/13 15:17, Bill Nottingham escribió: > > > Lukas Nykryn (lnyk...@redhat.com) said: > > >> --- > > >> TODO | 2 -- > > >> man/systemd.unit.xml.in

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-27 Thread Tollef Fog Heen
]] Cristian Rodríguez > El 26/03/13 15:17, Bill Nottingham escribió: > > Lukas Nykryn (lnyk...@redhat.com) said: > >> --- > >> TODO | 2 -- > >> man/systemd.unit.xml.in | 8 > >> src/core/condition.c | 16 +

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-26 Thread Cristian Rodríguez
El 26/03/13 15:17, Bill Nottingham escribió: Lukas Nykryn (lnyk...@redhat.com) said: --- TODO | 2 -- man/systemd.unit.xml.in | 8 src/core/condition.c | 16 src/core/condition.h |

Re: [systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-26 Thread Bill Nottingham
Lukas Nykryn (lnyk...@redhat.com) said: > --- > TODO | 2 -- > man/systemd.unit.xml.in | 8 > src/core/condition.c | 16 > src/core/condition.h | 1 + > src/core/load-fragment-gperf.gperf

[systemd-devel] [PATCH] condition: add option ConditionArchitecture

2013-03-26 Thread Lukas Nykryn
--- TODO | 2 -- man/systemd.unit.xml.in | 8 src/core/condition.c | 16 src/core/condition.h | 1 + src/core/load-fragment-gperf.gperf.m4 | 1 + 5 files changed, 26 insertions(+), 2 dele