New comment by CMB on void-packages repository

https://github.com/void-linux/void-packages/issues/18676#issuecomment-780135145

Comment:
Looks reasonable on the face of it, but a couple points:

* In the no-op softfail condition, I'd do something like an `exec pause` or `exec chpst ... pause`.  Otherwise runsv will just keep looping trying to start it.
* The command `xl list` pulls data from xenstored.  I'm not sure that it will return 0 before xenstored is up, so it's a chicken and egg problem.  I need to test that.  If `xl list` doesn't work, I know of other ways to test whether something is running in a Xen dom0.

Another thing I want to do with the xenstored service is have a configurable executable name.  There are two xenstored implementations shipped by Xen: the original C implementation (xenstored) and the newer ocaml implementation (oxenstored).  oxenstored is recommended by upstream and -- in my experience -- less buggy.  So xenstored/run should allow overriding the executable name in ./conf.

I'll send a well-tested PR for all of this.  It may be a day or two; I'm recovering from a 3-day power outage.