RFE for webrev: the meaning of -r
Jonathan Gibbons
Jonathan.Gibbons at Sun.COM
Tue Feb 24 15:21:22 UTC 2009
Neat looking tool.
Would it be too much of a stretch to have the tool optionally publish
the webrev to a remote location
such as the new code review server?
-- Jon
On Feb 24, 2009, at 6:57 AM, Jean-Christophe Collet wrote:
> I'll see what I can do
> In the meantime may I suggest you give jHg a try (http://sunweb.swiss/~jccollet/jHg/
> )
> It's a java tool I wrote that is much more flexible than the webrev
> script.
> I'm actually considering building a NetBeans module based on it.
>
> John Rose wrote:
>> On Feb 23, 2009, at 8:56 PM, Max (Weijun) Wang wrote:
>>
>>> Me too. I qgoto a patch and call 'webrev -N -r -2' to create a
>>> webrev now.
>>
>> Well, that's about what I do, except I have it merge up all my
>> queued changes sometimes.
>>
>> I sometimes have used a wrapper script which finds base revision of
>> the queue like this:
>>
>>> diffbase=$(hg log --rev qparent --template '{rev}')
>>> ...
>>> webrev -m -N -i $info -o $temp -r $diffbase -p "$parent" ...
>>
>> Saying "webrev -N -r qbase:qtip" could make sense...
>>
>> -- John
>
More information about the discuss
mailing list