diff options
| author | Alexandru Scvortov <alexandru@rabbitmq.com> | 2011-10-20 15:10:13 +0100 |
|---|---|---|
| committer | Alexandru Scvortov <alexandru@rabbitmq.com> | 2011-10-20 15:10:13 +0100 |
| commit | 386f5874b02712f1e5a943ddf2b73f092efdd5e6 (patch) | |
| tree | a4104270ac207c9598c6c9f4f444d464294cd0ea | |
| parent | df375c735ffcbbe6dead4327a0b499b6ca23e84f (diff) | |
| parent | 8dd2e7c274ef231f9625cb7066fb8fb546f7552a (diff) | |
| download | rabbitmq-server-386f5874b02712f1e5a943ddf2b73f092efdd5e6.tar.gz | |
merge bug24510 (make start-background-node start-rabbit-on-node breaks logging after bug 24332)
Also, don't fail if rabbit is already loaded.
| -rw-r--r-- | src/mirrored_supervisor_tests.erl | 1 | ||||
| -rw-r--r-- | src/rabbit.erl | 8 |
2 files changed, 7 insertions, 2 deletions
diff --git a/src/mirrored_supervisor_tests.erl b/src/mirrored_supervisor_tests.erl index ee9c7593..5e782a08 100644 --- a/src/mirrored_supervisor_tests.erl +++ b/src/mirrored_supervisor_tests.erl @@ -202,6 +202,7 @@ with_sups(Fun, Sups) -> Pids = [begin {ok, Pid} = start_sup(Sup), Pid end || Sup <- Sups], Fun(Pids), [kill(Pid) || Pid <- Pids, is_process_alive(Pid)], + timer:sleep(100), passed. start_sup(Spec) -> diff --git a/src/rabbit.erl b/src/rabbit.erl index 1c11f5af..530955c2 100644 --- a/src/rabbit.erl +++ b/src/rabbit.erl @@ -225,8 +225,12 @@ prepare() -> start() -> try %% prepare/1 ends up looking at the rabbit app's env, so it - %% needs to be loaded. - ok = application:load(rabbit), + %% needs to be loaded, but during the tests, it may end up + %% getting loaded twice, so guard against that + case application:load(rabbit) of + ok -> ok; + {error, {already_loaded, rabbit}} -> ok + end, ok = prepare(), ok = rabbit_misc:start_applications(application_load_order()) after |