Re: [PATCH 8/8] services: Add spice vdagent service.

David Craven skribis: >> If spice-vdagend produces a PID file, make sure to use #:pid-file here >> (there are several examples in the tree), which often provides more >> reliable startup notification. > > spice-vdagentd doesn't create a pid file itself. [0] [1] OK. In that

Re: [PATCH 8/8] services: Add spice vdagent service.

> If spice-vdagend produces a PID file, make sure to use #:pid-file here > (there are several examples in the tree), which often provides more > reliable startup notification. spice-vdagentd doesn't create a pid file itself. [0] [1] I haven't updated the documentation yet because I'm running

Re: [PATCH 8/8] services: Add spice vdagent service.

David Craven skribis: >> If spice-vdagend produces a PID file, make sure to use #:pid-file here >> (there are several examples in the tree), which often provides more >> reliable startup notification. > > I'm passing the -x flag which prevents spice-vdagentd from > daemonizing.

Re: [PATCH 8/8] services: Add spice vdagent service.

> If spice-vdagend produces a PID file, make sure to use #:pid-file here > (there are several examples in the tree), which often provides more > reliable startup notification. I'm passing the -x flag which prevents spice-vdagentd from daemonizing. I thought that make-forkexec-constructor took

Re: [PATCH 8/8] services: Add spice vdagent service.

> Also, I think this should be a separate patch as it seems to be unrelated. It's related to: > (service-extension profile-service-type + (compose list + spice-vdagent-configuration-spice-vdagent)) But ok. I'll do it first thing tomorrow =)