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