I think the change looks good and agree that it's probably a good idea to limit this to Session Init for now.
Should we have some tests for this? I'd think the following cases would be good to test: - User, no --confdir => loads all jobs from default paths - User, --confdir a => only loads jobs from "a" - User, --confdir a --confdir b => loads jobs from "a" and "b" - System, no --confdir => loads all jobs from default path - System, --confdir a => loads all jobs from "a" - System, --confdir a --confdir b => loads all jobs from "b" -- https://code.launchpad.net/~jamesodhunt/upstart/allow-multiple-cmdline-confdirs/+merge/156512 Your team Upstart Reviewers is requested to review the proposed merge of lp:~jamesodhunt/upstart/allow-multiple-cmdline-confdirs into lp:upstart. -- upstart-devel mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/upstart-devel
