Insights from bzr newbie coming from git

[Wouter van Heyst]

On Thu, Nov 30, 2006 at 09:06:34AM -0600, Matthew D. Fuller wrote:
> On Thu, Nov 30, 2006 at 05:01:29PM +0200 I heard the voice of
> Marius Kruger, and lo! it spake thus:
> > 
> > maybe add the following to all the commands taking revision
> > arguments: see also: `bzr help revisionspec`
> 
> .From a glance at a few of the revspec-using commands, the help for -r
> seems to have no description already.  perhaps just adding a "See help
> revisionspec" desc for them would do the trick (saying 'bzr' seems
> redundant).

Doh, that is the obvious place to point at the revisionspec topic. Can't
believe I missed that!

Still, I'm not entirely satisfied with the attached patch, comments
welcome.

`bzr help merge` (and the rest) now sports:

  -r ARG, --revision=ARG
                        See the revisionspec help topic for a list of valid
                        arguments

Wouter van Heyst
-------------- next part --------------
=== modified file 'bzrlib/option.py'
--- bzrlib/option.py	2006-10-31 20:56:47 +0000
+++ bzrlib/option.py	2006-11-30 16:22:47 +0000
@@ -236,7 +236,8 @@
                     'two values separated by a colon.')
 _global_option('profile',
                help='show performance profiling information')
-_global_option('revision', type=_parse_revision_str)
+_global_option('revision', type=_parse_revision_str,
+               help='See the revisionspec help topic for a list of valid arguments')
 _global_option('show-ids', 
                help='show internal object ids')
 _global_option('timezone', 

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 307 bytes
Desc: Digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20061130/d0abe75c/attachment.pgp