I've changed the following: - I've removed the reread, avail, and reread commands from supervisorctl as they're not quite yet ready for primetime.
- I removed the backend code from rpcinterface to handle reread, as it only computes a diff rather than restarting any processes. - I've changed "drop" to "remove" in supervisorctl. - I've changed the way errors are handled for the newly added supervisorctl commands. Basically, we're left with "add" and "remove" as user-visible and XML-RPC API commands. Which is fine. When we get up the gumption to finish reread, we'll do that. Thanks again, - C Anders Quist wrote: > On Fri, 25 Jul 2008 08:50:14 -0400, Chris McDonough <[EMAIL PROTECTED]> > writes: > >> Looks good, Anders! Thank you (esp. for the tests). I will apply it >> to the trunk today. > > You're welcome. It's always nice to be able to contribute to FOSS on > paid time :) > >> Would you mind if I renamed the "drop" supervisorctl command to >> "remove"? > > I don't have a strong oppinion on the matter. I chose drop over remove > mostly because it felt more natural that you could re-add something > that had been dropped, whereas remove felt too irreversible. > > [snip] > _______________________________________________ Supervisor-users mailing list [email protected] http://lists.supervisord.org/mailman/listinfo/supervisor-users
