Bazaar

Merge lp:~jbowtie/bzr/fix-555439 into lp:bzr

  1. fix-555439
  2. Merge into bzr.dev
Proposed by John C Barstow
Status: Merged
Approved by: Martin Pool
Approved revision: no longer in the source branch.
Merged at revision: 5208
Proposed branch: lp:~jbowtie/bzr/fix-555439
Merge into: lp:bzr
Diff against target: 57 lines (+22/-16)
2 files modified
bzrlib/builtins.py (+1/-0)
bzrlib/help_topics/__init__.py (+21/-16)
To merge this branch: bzr merge lp:~jbowtie/bzr/fix-555439
Reviewer Review Type Date Requested Status
Aaron Bentley (community) Approve
Martin Pool Approve
Review via email: mp+24289@code.launchpad.net

Commit message

(jbowtie) document additional bzr environment variables

To post a comment you must log in.
Revision history for this message
Martin Pool (mbp) wrote :

it might be nice to give a mini-example but this is an improvement.

eventually it might be nice to not repeat ourselves and just have some kind of global definition of variables we care about. it may not be worth it if it's just once in the code and once in the docs.

review: Approve
Revision history for this message
Aaron Bentley (abentley) wrote :

I think we should use consistent quotation marks within the help text. Have we started using curly quotes? If so, we should at least use them consistently in this string.

review: Needs Fixing
Revision history for this message
Martin Pool (mbp) wrote :

On 30 April 2010 10:19, Aaron Bentley <email address hidden> wrote:
> Review: Needs Fixing
> I think we should use consistent quotation marks within the help text.

agree.

> Have we started using curly quotes?  If so, we should at least use them consistently in this string.

I didn't spot that. I don't think using curly quotes in help text is
worth the unicode hassles.

--
Martin <http://launchpad.net/~mbp/>

Revision history for this message
John C Barstow (jbowtie) wrote :

> I think we should use consistent quotation marks within the help text. Have
> we started using curly quotes? If so, we should at least use them
> consistently in this string.

Sorry, I didn't spot that either; I cut and paste that particular string.

Revision history for this message
John C Barstow (jbowtie) wrote :

I've just pushed a fix for the quotes, turning them into single straight quotes like the surrounding material.

Revision history for this message
Aaron Bentley (abentley) :
review: Approve
Revision history for this message
Vincent Ladeuil (vila) wrote :

Looks like all we need now is the contributor agreement: http://www.canonical.com/contributors

Can you give it a look ?

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'bzrlib/builtins.py'
--- bzrlib/builtins.py 2010-04-28 10:33:44 +0000
+++ bzrlib/builtins.py 2010-04-30 01:42:31 +0000
@@ -4681,6 +4681,7 @@
46814681
4682class cmd_bind(Command):4682class cmd_bind(Command):
4683 __doc__ = """Convert the current branch into a checkout of the supplied branch.4683 __doc__ = """Convert the current branch into a checkout of the supplied branch.
4684 If no branch is supplied, rebind to the last bound location.
46844685
4685 Once converted into a checkout, commits must succeed on the master branch4686 Once converted into a checkout, commits must succeed on the master branch
4686 before they will be applied to the local branch.4687 before they will be applied to the local branch.
46874688
=== modified file 'bzrlib/help_topics/__init__.py'
--- bzrlib/help_topics/__init__.py 2010-03-02 10:21:39 +0000
+++ bzrlib/help_topics/__init__.py 2010-04-30 01:42:31 +0000
@@ -591,22 +591,27 @@
591_env_variables = \591_env_variables = \
592"""Environment Variables592"""Environment Variables
593593
594================ =================================================================594=================== =======================================================================
595BZRPATH Path where bzr is to look for shell plugin external commands.595BZRPATH Path where bzr is to look for shell plugin external commands.
596BZR_EMAIL E-Mail address of the user. Overrides EMAIL.596BZR_EMAIL E-Mail address of the user. Overrides EMAIL.
597EMAIL E-Mail address of the user.597EMAIL E-Mail address of the user.
598BZR_EDITOR Editor for editing commit messages. Overrides EDITOR.598BZR_EDITOR Editor for editing commit messages. Overrides EDITOR.
599EDITOR Editor for editing commit messages.599EDITOR Editor for editing commit messages.
600BZR_PLUGIN_PATH Paths where bzr should look for plugins.600BZR_PLUGIN_PATH Paths where bzr should look for plugins.
601BZR_HOME Directory holding .bazaar config dir. Overrides HOME.601BZR_DISABLE_PLUGINS Plugins that bzr should not load.
602BZR_HOME (Win32) Directory holding bazaar config dir. Overrides APPDATA and HOME.602BZR_PLUGINS_AT Plugins to load from a directory not in BZR_PLUGIN_PATH.
603BZR_REMOTE_PATH Full name of remote 'bzr' command (for bzr+ssh:// URLs).603BZR_HOME Directory holding .bazaar config dir. Overrides HOME.
604BZR_SSH Path to SSH client, or one of paramiko, openssh, sshcorp, plink.604BZR_HOME (Win32) Directory holding bazaar config dir. Overrides APPDATA and HOME.
605BZR_LOG Location of .bzr.log (use '/dev/null' to suppress log).605BZR_REMOTE_PATH Full name of remote 'bzr' command (for bzr+ssh:// URLs).
606BZR_LOG (Win32) Location of .bzr.log (use 'NUL' to suppress log).606BZR_SSH Path to SSH client, or one of paramiko, openssh, sshcorp, plink.
607BZR_COLUMNS Override implicit terminal width.607BZR_LOG Location of .bzr.log (use '/dev/null' to suppress log).
608BZR_CONCURRENCY Number of processes that can be run concurrently (selftest).608BZR_LOG (Win32) Location of .bzr.log (use 'NUL' to suppress log).
609================ =================================================================609BZR_COLUMNS Override implicit terminal width.
610BZR_CONCURRENCY Number of processes that can be run concurrently (selftest).
611BZR_PROGRESS_BAR Override the progress display. Values are 'none', 'dots', or 'tty'.
612BZR_PDB Control whether to launch a debugger on error.
613BZR_SIGQUIT_PDB Control whether SIGQUIT behaves normally or invokes a breakin debugger.
614=================== =======================================================================
610"""615"""
611616
612617