Created attachment 63611 [details] [review]
<servicehelper> is not supported in the included config file, i. e. it's not merged in merge_included().
There's clearly no reason it shouldn't be supported in the included config file along with <user>, <type> and others. It's quite reasonable for a client willing to override the default servicehelper, e. g. in system-local.conf.
Is there anything that stops my proposed patch from being pulled into trunk?
I don't think it's sensible to override the service launching helper (which is security-sensitive and does not have a "public API"), but for completeness, OK.
Looks like I merged this already, in fact. Fixed in git for 1.7.0