[merge] bzr whoami
Robey Pointer
robey at lag.net
Tue Jul 4 23:05:19 BST 2006
On 3 Jul 2006, at 7:12, John Arbash Meinel wrote:
>>> v- in general, I prefer 'run_bzr' to runbzr. I personally think
>>> runbzr
>>> should go away. (It only exists on ExternalBase, not
>>> TestCaseInTempDir).
>>
>> I just missed that one when migrating the rest. I do think that it's
>> utterly confusing which one of the various run*bzr* methods is
>> meant to
>> be called. Why not mark the other ones as deprecated? I picked
>> 'run_bzr' pretty much by personal choice since I couldn't remember
>> which
>> one was preferred from the last time the 'whoami' patch failed to
>> land.
>
> I agree, which is why I think run_bzr should die. :)
> But it is used in enough code that doing a cleanup isn't very trivial.
> And they do have slightly different semantics (runbzr() splits on
> whitespace).
Wait, are you sure you mean 'run_bzr'? :)
'runbzr' is just a wrapper around run_bzr, using a fragile arg-
parsing strategy and not supporting things like encoding. If
anything, I think that's the one that should go...
>>> I think we may want to validate that the supplied value really is a:
>>>
>>> "Full Name <email at address.com>"
>>>
>>> Rather than just letting them do:
>>>
>>> bzr whoami george
>>>
>>> We do accept short names (I don't think we explicitly require email
>>> addresses anywhere). But we would probably at least want to warn the
>>> user that it is recommended to use a full email address, etc.
>>
>> I think that's a good idea. I would want to just reuse the method
>> that
>> does this extraction in config, so it should be a separate patch.
>
> Sure. I realize you have a branch for this, but is there any reason
> you
> didn't post a patch?
Sorry, I just wanted to get the patch landed. I've had terrible luck
in the past with patches getting lost, either because of long, side-
tracked debates or just getting lost in the shuffle.
Also I thought it would be harder than it turned out to be. I think
the tiny attached bundle is all that's required. (It's also in my
whoami branch.)
robey
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: whoami-bundle.txt
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20060704/6f2483e4/attachment.txt
More information about the bazaar
mailing list