RFR (S): CODETOOLS-7901317: Add flag to disable comments in webrev

Bengt Rutisson bengt.rutisson at oracle.com
Mon Mar 2 16:40:28 UTC 2015


Hi everyone,

I've been missing the possibility to produce a webrev that does not 
include the comments from the changesets. I often use mq to track a set 
of changes and when I create a webrev I normally don't want all the 
names of my mq patches to appear in the webrev.

Tom Rodrigues once had a version of webrev where -C meant that no 
comments were includes. I've added this flag to the offical webrev script.

Using this review request as an examle. Here is a version with comments:
http://cr.openjdk.java.net/~brutisso/7901317/webrev.00-with-comment/

Here's the same without comment (so produced with "$ ksh webrev.ksh -C":
http://cr.openjdk.java.net/~brutisso/7901317/webrev.00-without-comment/

Bug report:
https://bugs.openjdk.java.net/browse/CODETOOLS-7901317

I need a sponsor for this fix as I am not a committer in the Code-tools 
project.

Thanks,
Bengt


More information about the webrev-dev mailing list