Created attachment 63611 [details] [review] Proposed patch <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
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.