Mike Kupfer writes:
> >>>>> "Rich" == Richard Lowe <richlowe at richlowe.net> writes:
>
> Rich> However, at present, it outright doesn't work (errors from Hg,
> Rich> because the parent (webrev) isn't a workspace).
>
> I don't understand. If you're using -p <path_to_prev_webrev>, it should
> be -p <path_to_prev_snapshot>.
webrev(1) documents otherwise:
INCREMENTAL REVIEWS
When conducting multiple rounds of code review, it may be
desirable to generate a webrev which represents the delta
between reviews. In this case, set the parent workspace to
the path to the old webrev:
$ webrev -o ~/public_html/myreview-rd2/ \
-p ~/public_html/myreview/
(I was surprised to see it, too.)
--
James Carlson, Solaris Networking <james.d.carlson at sun.com>
Sun Microsystems / 35 Network Drive 71.232W Vox +1 781 442 2084
MS UBUR02-212 / Burlington MA 01803-2757 42.496N Fax +1 781 442 1677
