Re: [systemd-devel] [PATCH] Add Frugalware prefdm service

2011-03-15 Thread Lennart Poettering
On Fri, 11.03.11 01:11, Miklos Vajna (vmik...@frugalware.org) wrote: > On Fri, Mar 11, 2011 at 12:56:58AM +0100, Lennart Poettering > wrote: > > On Thu, 10.03.11 23:04, Miklos Vajna (vmik...@frugalware.org) wrote: > > > > > +[Service] > > > +EnvironmentFile=/etc/sysconfig/desktop > > > +ExecSta

Re: [systemd-devel] [PATCH] Add Frugalware prefdm service

2011-03-10 Thread Miklos Vajna
On Fri, Mar 11, 2011 at 12:56:58AM +0100, Lennart Poettering wrote: > On Thu, 10.03.11 23:04, Miklos Vajna (vmik...@frugalware.org) wrote: > > > +[Service] > > +EnvironmentFile=/etc/sysconfig/desktop > > +ExecStart=$desktop > > I don't think this works. Don't worry, I tested it before submitti

Re: [systemd-devel] [PATCH] Add Frugalware prefdm service

2011-03-10 Thread Lennart Poettering
On Thu, 10.03.11 23:04, Miklos Vajna (vmik...@frugalware.org) wrote: > +[Service] > +EnvironmentFile=/etc/sysconfig/desktop > +ExecStart=$desktop I don't think this works. You cannot use a variable for the binary path, due to some implications with SELinux. If you want to do it nonetheless use so

Re: [systemd-devel] [PATCH] Add Frugalware prefdm service

2011-03-10 Thread Miklos Vajna
On Thu, Mar 10, 2011 at 10:29:47PM +0100, Lennart Poettering wrote: > Hmm, prefdm is not supposed to be standardized name. It's something > fedora and mandriva specific, and hence stored in the distro-specific > dirs. Exactly, that's why I added the service file in the corresponding subdir, so o

Re: [systemd-devel] [PATCH] Add Frugalware prefdm service

2011-03-10 Thread Lennart Poettering
On Wed, 09.03.11 01:04, Miklos Vajna (vmik...@frugalware.org) wrote: > Hi, > > The mandrive patches made this local patch fail to apply - which > reminded me to submit the attached patch. :) Hmm, prefdm is not supposed to be standardized name. It's something fedora and mandriva specific, and hen

[systemd-devel] [PATCH] Add Frugalware prefdm service

2011-03-08 Thread Miklos Vajna
Hi, The mandrive patches made this local patch fail to apply - which reminded me to submit the attached patch. :) It does not affect other distros and adds the same prefdm.service we were adding manually in the package build script. Thanks. PS: Yes, in the long run we want to migrate to separat