Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Add appmon dependency, remove '-mode embedded' #2

Merged
merged 1 commit into from Feb 4, 2013

Conversation

Projects
None yet
2 participants
Contributor

slfritchie commented Feb 4, 2013

Hi, Loïc, I've a couple of small patches here that IMHO are useful.

  1. Add a dependency on the appmon app. Then appmon:start(). can be used to help see the supervisory processes tree for the elevators app as well as sasl and kernel.
  2. Remove the -mode embedded flag from the erl command line. Using embedded mode makes it difficult to load new modules, e.g., if someone is experimenting with some new elevator code and coincidentally to use a new module.

Also, the embedded mode is not included in DOS/Win elevators.cmd version of the script.

essen added a commit that referenced this pull request Feb 4, 2013

Merge pull request #2 from slfritchie/add-appmon
Add appmon dependency, remove '-mode embedded'

@essen essen merged commit 68f9562 into extend:master Feb 4, 2013

Owner

essen commented Feb 4, 2013

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment