[rabbitmq-discuss] Rabbit Boot Steps

Matthew Sackman matthew at rabbitmq.com
Wed Jun 22 11:32:20 BST 2011


Hi Alvaro,

On Sun, Jun 19, 2011 at 11:46:04PM +0200, Alvaro Videla wrote:
> I've extracted the boot steps logic into its own module. I think in
> this way it might get easier to use by projects outside RabbitMQ.
> 
> So far it depends on rabbit_misc:all_module_attributes/1 and
> rabbit_misc:build_acyclic_graph/3 but they are used in other parts of
> Rabbit's code so I don't know what's the best approach here.
> 
> I've added some specs, I hope they are correct, since is the first
> time I write those. Also I added extra functionality. So it's possible
> to have boot steps like -my_app_boot_step and so on. In this way, the
> prefix can be customized. There's a function boot_steps/1 that will
> look for the <prefix>_boot_step attributes. If prefix is omitted, then
> the current application name will be used.
> 
> My idea is to extract this module to some library via a Make target in
> the same way rabbit_common is extracted today.
> 
> I've attached the patch and the new module, let me know what you
> think.

Many thanks for your efforts.

Because the functionality is now completely non rabbit-specific, they
should not really be in a module with a rabbit_ prefix. The more
reusable parts of rabbit's code base tend to follow this rule.

We'd probably need you to sign a contributor agreement to accept these
changes, which I'll send to you off list - I don't think you've already
signed one.

Matthew


More information about the rabbitmq-discuss mailing list