RFE for webrev: the meaning of -r

Jean-Christophe Collet Jean-Christophe.Collet at Sun.COM
Tue Feb 24 15:37:17 UTC 2009


Jonathan Gibbons wrote:
> 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?
>

Good point. I'll try to do that as soon as I get some time

> -- 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