Simon,<div><br></div><div>thanks for the changes.</div><div><br></div><div>We made other changes that would be good if they could be merged:</div><div>- if configuration file is specified and node is not specified it could</div>
<div> allow strange behaviors, we thought it would be nice if it would look</div><div> for a "default" node in the configuration file.</div><div> Please note that this is done only if the configuration file is given,</div>
<div> else it would always look for the default config file and node "default"</div><div> if default config files would be present.</div><div>- the default vhost "/" (line 517 of your version) was not configurable.</div>
<div> vhost is treated differently for list and declare, the current situation</div><div> does not allow to define a vhost in the configuration file.</div><div> Adding a default_vhost to the configuration file allow the possibility</div>
<div> to configure every parameter from the configuration file independently</div><div> from the sub command executed. (which is not the case in the current version)</div><div>- die was printing messages without going to new line</div>
<div><br></div><div>Updated script is available at:</div><div><a href="https://github.com/mpaladin/rabbitmq-custom/blob/master/rabbitmqadmin/rabbitmqadmin" target="_blank">https://github.com/mpaladin/rabbitmq-custom/blob/master/rabbitmqadmin/rabbitmqadmin</a><br clear="all">
<br></div><div>Cheers,</div><div>---<br>Massimo Paladin<br><br>email: <a href="mailto:massimo.paladin@gmail.com" target="_blank">massimo.paladin@gmail.com</a><br>website: <a href="http://www.mpaladin.com/" target="_blank">http://www.mpaladin.com</a><br>
flickr's page: <a href="http://flickr.com/photos/massimop" target="_blank">http://flickr.com/photos/massimop</a><br>
<br><br><div class="gmail_quote">On Mon, Feb 13, 2012 at 4:54 PM, Simon MacMullen <span dir="ltr"><<a href="mailto:simon@rabbitmq.com" target="_blank">simon@rabbitmq.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>On 10/02/12 19:06, Massimo Paladin wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Now it tries to read from ~/.rabbitmqadmin.conf if node is specified and<br>
the config file is not passed.<br>
</blockquote>
<br></div>
Thanks. I've merged your changes (+ some more tweaks).<div><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I am going to look at bash completion soon.<br>
</blockquote>
<br></div>
Of course that's a nice-to-have :-) I don't know if anyone apart from me uses rabbitmqadmin bash completion...<div><div><br>
<br>
Cheers, Simon<br>
<br>
-- <br>
Simon MacMullen<br>
RabbitMQ, VMware<br>
</div></div></blockquote></div><br></div>