[MERGE] Prompt for user names if they are not in the configuration.
Jelmer Vernooij
jelmer at samba.org
Thu Apr 9 21:27:20 BST 2009
Hi Vincent,
Vincent Ladeuil wrote:
>>>>>> "Jelmer" == Jelmer Vernooij <jelmer at vernstok.nl> writes:
>>>>>>
>
> Jelmer> Jelmer Vernooij wrote:
> >> The attached patch makes bzr prompt for user names when it needs to
> >> authenticate but there is no username known, similar to what's already done
> >> for passwords.
> >>
>
> Jelmer> Updated patch with changes requested by Vincent: get_user() now defaults
> Jelmer> to returning getpass.getuser(), to match current behaviour unless
> Jelmer> explicitly requested (as is done for http) in which case it will prompt
> Jelmer> the user.
>
> This sounds good except for the missing tests that http will
> indeed prompt for a user and that the other transports won't
> (sftp and ftp will be enough for the "other" and even sftp...).
>
> A patch against bzr.dev once your submission for get_username is
> merged will be clearer.
>
The attached patch adds these tests. Please let me know what you think.
Cheers,
Jelmer
-------------- next part --------------
A non-text attachment was scrubbed...
Name: username.diff
Type: text/x-patch
Size: 28919 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090409/08f61325/attachment-0001.bin
More information about the bazaar
mailing list