Re: [PATCH] svn --help to include HEAD, BASE, etc for -r

Martin Pool <mbp@samba.org> writes:

> Index: main.c
> ===================================================================
> --- main.c (revision 4083)
> +++ main.c (working copy)
> @@ -65,7 +65,14 @@
> {"recursive", 'R', 0, "descend recursively"},
> {"nonrecursive", 'N', 0, "operate on single directory only"},
> {"revision", 'r', 1,
> - "revision ARG, X:Y range ({ARG}, {X}:{Y} for dates)"},
> + "revision X or X:Y range,\n"
> + " {DATE} for dates,\n"
> + " HEAD latest in repository,\n"
> + " BASE basis of working copy,\n"

We have to be careful here, it's important that we are accurate. A
working copy does not have *a* BASE revision in general, as it may be
a mixed revision working copy. I would be happier if you referred to
something like "base revision of path" and did not imply that it
referred to the working copy as a whole.

> + " COMMITTED last revision of last change,\n"

Too many lasts.

> + " PREV revision before last change"
> + /* spacing corresponds to svn_opt_format_option */
> + },
> {"file", 'F', 1, "read data from file ARG"},
> {"incremental", svn_cl__incremental_opt, 0,
> "give output suitable for concatenation"},

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org