WIP Implement service dependency management

shosti commented 9 years ago

This is a preliminary attempt at fixing #14—I just wanted to validate the approach and get some feedback. It hasn't been thoroughly tested yet. It should allow for services to wait for other services to enter a ready state.

To do:

[ ] Proper handling of stopping services
[ ] Prettify the :depends-on attribute—currently it requires a form of '(("Foo" running) ("Bar" ready)) and so forth, but it should probably default to 'running if it's just a string instead of a list
[ ] Proper integration tests
[ ] Handling custom states (other than 'running and 'ready)—I'm not quite sure what the best approach here is

rejeep commented 9 years ago

:+1: Will take a look when I get some time!

rejeep commented 9 years ago

I think it looks great! Two things come to mind:

When you make the non WIP-commits, can you make a separate commit when you move the code from prodigy-start-service to the other functions? Otherwise it's hard to see if something changed in the big chunk of code that was moved.
Does this work with :init-async?

DamienCassou commented 2 years ago

I'm closing all old PRs. If you still care, please rebase your branch and reopen the PR.

rejeep / prodigy.el

WIP Implement service dependency management #66