You are correct. Sorry (wrong -r option and use of a file list misled me into thinking it was generating the right content).
I have added http://cr.openjdk.java.net/~mduigou/JDK-8030350/3/webrev/ and checked to make sure it has the right content. Mike On Jan 8 2014, at 03:57 , Magnus Ihse Bursie <magnus.ihse.bur...@oracle.com> wrote: > On 2014-01-08 02:09, Mike Duigou wrote: >> Hello all; >> >> I have added the BSD supported requested in previous reviews and I believe >> this change is ready to go in. >> >> http://cr.openjdk.java.net/~mduigou/JDK-8030350/2/webrev/ >> >> I need a sponsor for the hotspot repo portion of the change. >> >> I've also generated build log diffs to illuminate the additional warnings >> produced and will file bugs shortly for a few of the cases. >> >> Mike > > Mike, > > There seems to be a problem with the webrev - there are no actual changes > recorded in it. > > /Magnus