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