Hi Jason, jason kirtland wrote: > supervisorctl -s unix://somepath was bombing with an invalid URL > exception. Attached patch fixes the issue and adds a test. The error > is only triggered in the command line parsing code path, so the test > added is a little different than its config-file-parsing test > neighbors. Could be there's a better home for the test that I missed?
Thanks for this patch. I've committed a slightly modified version that adds support for the "unix" scheme but not "file". We don't support "file" elsewhere in the codebase. Your unit test is fine (and thanks for it!). We should have had some tests for this method in the test_datatypes module but that's our fault. I've added a few. Regards, Mike -- Mike Naberezny Maintainable Software http://maintainable.com _______________________________________________ Supervisor-users mailing list [email protected] http://lists.supervisord.org/mailman/listinfo/supervisor-users
