/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to doc/developers/missing.txt

  • Committer: Vincent Ladeuil
  • Date: 2009-04-27 16:10:10 UTC
  • mto: (4310.1.1 integration)
  • mto: This revision was merged to the branch mainline in revision 4311.
  • Revision ID: v.ladeuil+lp@free.fr-20090427161010-7swfzeagf63cpixd
Fix bug #367726 by reverting some default user handling introduced
while fixing bug #256612.

* bzrlib/transport/ssh.py:
(_paramiko_auth): Explicitly use getpass.getuser() as default
user.

* bzrlib/transport/ftp/_gssapi.py:
(GSSAPIFtpTransport._create_connection): Explicitly use
getpass.getuser() as default user.

* bzrlib/transport/ftp/__init__.py:
(FtpTransport._create_connection): Explicitly use
getpass.getuser() as default user.

* bzrlib/tests/test_sftp_transport.py:
(TestUsesAuthConfig.test_sftp_is_none_if_no_config)
(TestUsesAuthConfig.test_sftp_doesnt_prompt_username): Revert to
None as the default user.

* bzrlib/tests/test_remote.py:
(TestRemoteSSHTransportAuthentication): The really offending one:
revert to None as the default user.

* bzrlib/tests/test_config.py:
(TestAuthenticationConfig.test_username_default_no_prompt): Update
test (and some PEP8).

* bzrlib/smtp_connection.py:
(SMTPConnection._authenticate): Revert to None as the default
user.

* bzrlib/plugins/launchpad/account.py:
(_get_auth_user): Revert default value handling.

* bzrlib/config.py:
(AuthenticationConfig.get_user): Fix doc-string. Leave default
value handling to callers.

Show diffs side-by-side

added added

removed removed

Lines of Context:
doc/developers/missing.txt
 
1
Missing
 
2
=======
 
3
 
 
4
Missing is used to find out the differences between the current branch and
 
5
another branch.
 
6
 
 
7
The performance analysis itself brings no further points than the
 
8
incremental-push-pull one.
 
9
 
 
10
More importantly, the UI have been considered not optimal: missing finds and
 
11
displays the differences between two branches, presenting the revisions that
 
12
are not common to both branches as two sets:
 
13
 
 
14
* the revisions that are present only in the current branch,
 
15
* the revisions that are present only in the other branch.
 
16
 
 
17
A quick and dirty survey indicates that most of the users are interested in
 
18
only one set of revisions at a time.
 
19
 
 
20
From a performance point of view, it may be more appropriate to calculate only
 
21
the set the user is asking for.
 
22
 
 
23
It has been proposed that the missing command be deprecated in favor of a
 
24
--dry-run option for the push, pull, merge commands.
 
25
 
 
26
In the mean time, the missing command stays interesting as it provides an easy
 
27
way to test, measure and optimize graph differences processing.