[MERGE] Make command discovery entirely callback driven.
Ian Clatworthy
ian.clatworthy at internode.on.net
Mon Jun 15 09:06:40 BST 2009
Robert Collins wrote:
> On Mon, 2009-06-15 at 17:03 +1000, Ian Clatworthy wrote:
>>
>>>>>>> +def get_all_cmds():
>>>>>>> d = _builtin_commands()
>>>>>>> if plugins_override:
>> So I'm trying to say that this variable is still used in the code I
>> reviewed, but isn't declared as a parameter any more. Maybe you've
>> changed the code since ...
>
> Oh! I get it now. Changed my mind in some code changes; I'll put the
> parameter back as the easiest way to fix.
:-)
>>> remaining issues:
>>> - do we need to warn people off from using the hooks
>> We don't need to warn them off - just let them know that they aren't
>> *required* to use them to register a command in a plugin.
>
> Ok. I'm not sure how best to get the message across though:- do you
> think just mentioning it in the docs for (say) get_command is enough?
Yep - that's fine.
Ian C.
More information about the bazaar
mailing list