[Pkg-xen-devel] [PATCH 08/13] xen init script: rewrite xenstored start logic

[Hans van Kranenburg]

On 2/12/19 7:14 PM, Ian Jackson wrote:
> Hans van Kranenburg writes ("Re: [PATCH 08/13] xen init script: rewrite xenstored start logic"):
>> So the question we should answer first is: What do we allow the user to
>> set as value? /bin/bash? And how should the init script deal with that?
> 
> If they specify an absolute path, it should be used.  Eg,
>   /home/alice/Xen/xen/tools/xenstore/xenstore

Ok.

> If the user specifies /bin/bash they get to keep all the resulting
> pieces.
> 
>> Or do you want to be able to use this to run your own xenstored
>> replacement implementation from /usr/local/ian/ixenstored ?
> 
> Right.  Or a wrapper script or something (although s-s-d will
> mishandle that).

Yes, I'm no start-stop-daemon expert, but this should be well tested in
combination with the already-running check.

Having the init script wrongly start another xenstored just makes
everything hang withouta good explanation to the user.

K