summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Commentbug24919Simon MacMullen2012-05-231-0/+3
|
* on a second thought, it is better to reject options with missing argumentsFrancesco Mazzoli2012-05-232-6/+1
|
* incorporate most of Simon's patch changesFrancesco Mazzoli2012-05-224-129/+74
| | | | | | | It's everything but the removal of global options and the possibility of providing just the command atom instead of {command, opts}, + 2 bug fixes in the patch code (returning `bad_argument' when there is an option with no argument and reversing the arguments).
* get_options => parse_argumentsFrancesco Mazzoli2012-05-214-56/+59
|
* convert to list at the beginningFrancesco Mazzoli2012-05-211-2/+5
|
* simplify `rabbit_misc:get_options/4'Francesco Mazzoli2012-05-214-98/+84
| | | | | | | | | | | Incidentally, it is also much more efficient (not that it matters). I will probably rename it to something like `parse_arguments'. `CommandOpts' and `GlobalOpts' could be one, but I think that `GlobalOpts' is really useful since adding the global options to each single command is a pain. We could, obviously, put them together in a tuple, but then it's not that different from two separate arguments.
* oopsFrancesco Mazzoli2012-05-171-1/+1
|
* add test for `rabbit_misc:get_options/4'Francesco Mazzoli2012-05-173-26/+88
|
* change to how `get_options/4' handles flagsFrancesco Mazzoli2012-05-173-36/+49
| | | | | Now it doesn't complain on flags that should not be there, it simply eats only the flags relevant to the command.
* added checks to `rabbit_misc:get_options/3'.Francesco Mazzoli2012-05-163-76/+175
| | | | | See comments on top of the new definition (now its arity is 4). I still need to update the tests.
* merge defaultFrancesco Mazzoli2012-05-169-68/+69
|\
| * refactorMatthias Radestock2012-05-091-10/+3
| |
| * check for a new pid when registered children die in mirrored_supervisor_teststim2012-05-091-1/+1
| |
| * do not kill the child explicitlytim2012-05-081-1/+1
| |
| * for bug24362 - assert that worker name is unregistered by the time we get a ↵Tim Watson2012-05-081-6/+19
| | | | | | | | 'DOWN' notification
| * pmon is very much hipe-worthy.Simon MacMullen2012-05-041-1/+1
| |
| * Merged bug24889 into defaultEmile Joubert2012-05-0422-62/+558
| |\
| | * cosmeticMatthias Radestock2012-05-041-9/+5
| | |
| | * cosmeticMatthias Radestock2012-05-041-2/+2
| | |
| | * Merge bug24729 (again)Simon MacMullen2012-05-035-33/+24
| | |\
| | | * Merge bug 24901Simon MacMullen2012-05-031-2/+1
| | | |\
| | | | * Reorder exception contentsbug24901Emile Joubert2012-04-231-2/+1
| | | | |
| | | * | Revert 9d59000c4a44, 38eb4652c7e3, f6d1609212f9, e1311dd42eba (timeout ↵Simon MacMullen2012-05-031-2/+2
| | | | | | | | | | | | | | | | | | | | changes for mirrored_sup tests)
| | | * | Revert aa8a9e4d1bb2 (extra debugging for mirrored_supervisor)Simon MacMullen2012-05-031-10/+8
| | | | |
| | | * | cosmeticMatthias Radestock2012-05-032-27/+19
| | | | |
| | | * | Fix for R12B-3 that was broken by bug 24196.Simon MacMullen2012-05-011-2/+2
| | | | |
| | | * | A bit more debugging information on why we might get an error back from the ↵Simon MacMullen2012-05-011-8/+10
| | | | | | | | | | | | | | | | | | | | delegate.
| | * | | Fiddle with commentsbug24729Simon MacMullen2012-05-031-6/+7
| | | | |
| | * | | Removed unused types, added dummy -spec to silence compilerFrancesco Mazzoli2012-05-012-5/+7
| | | | |
| | * | | merge defaultFrancesco Mazzoli2012-05-011-1/+4
| | |\ \ \ | | | |/ / | | |/| |
| | | * | Add comment explaining why ack() and state() exist.Francesco Mazzoli2012-04-241-1/+4
| | | | |
| | | * | merge defaultFrancesco Mazzoli2012-04-2315-223/+262
| | | |\ \
| * | | | | Display things that are strings as strings rather than terms, since Erlang's ↵bug24889Simon MacMullen2012-04-271-14/+16
| | | | | | | | | | | | | | | | | | | | | | | | string-detection algorithm breaks on non-ASCII characters. Assume that ~p's use of quotation marks was a deliberate aesthetic choice.
* | | | | | `proplists:get_bool/2' does not do what I think it did, using `lookup/2'Francesco Mazzoli2012-05-091-6/+4
| | | | | |
* | | | | | oops, had left io:format in the codeFrancesco Mazzoli2012-05-031-2/+1
| | | | | |
* | | | | | fix typo and opt checkingFrancesco Mazzoli2012-05-011-2/+5
| | | | | |
* | | | | | use pp flag isntead of re-writing the flagFrancesco Mazzoli2012-05-011-4/+4
| | | | | |
* | | | | | Check that options are not used in the wrong placeFrancesco Mazzoli2012-05-011-0/+15
| |/ / / / |/| | | |
* | | | | Merge bug 24196.Simon MacMullen2012-05-0128-164/+411
|\ \ \ \ \
| * | | | | Log errors when children fail to start.Simon MacMullen2012-04-301-7/+7
| | | | | |
| * | | | | Merge headsSimon MacMullen2012-04-300-0/+0
| |\ \ \ \ \
| | * | | | | Added tag rabbitmq_v2_8_2 for changeset 4136141f70b9Simon MacMullen2012-04-300-0/+0
| | | | | | |
| * | | | | | another attempt to track down the mirrored_supervisor_tests CI failureMatthias Radestock2012-04-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | waiting forever is all very well...except when we *expect* a failure
| * | | | | | cosmeticMatthias Radestock2012-04-271-3/+3
| | | | | | |
| * | | | | | let's wait foreverMatthias Radestock2012-04-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ...so we can investigate what is going since this fails in CI even with a 1 minute timeout
| * | | | | | bug: code_change/3 should return {ok, State}Matthias Radestock2012-04-271-1/+1
| |/ / / / /
| * | | | | Merge bug24911rabbitmq_v2_8_2Simon MacMullen2012-04-271-1/+1
| |\ \ \ \ \
| | * | | | | another attempt at pacifying the CI godsMatthias Radestock2012-04-271-1/+1
| | |/ / / /
| * | | | | properly ignore dynamic children that return 'ignore' from init/1bug24911Matthias Radestock2012-04-271-0/+4
| |/ / / / | | | | | | | | | | | | | | | | | | | | thus preventing an explosion on supervisor shutdown when we would otherwise trip over unlinking an 'undefined' pid.
| * | | | Merge bug24909Simon MacMullen2012-04-261-3/+3
| |\ \ \ \
View Lorry Controller Status