Please review the updated webrev @ http://cr.openjdk.java.net/~sundar/8012975/webrev.01/

Thanks
-Sundar

On Friday 03 May 2013 02:56 PM, Alan Bateman wrote:
On 03/05/2013 07:47, A. Sundararajan wrote:

Thanks. Looks like the first one has not been removed. But second one was removed: hg stat shows

R make/sun/org/mozilla/javascript/Makefile

(also the webrev shows it as removed). Perhaps patch does not take care of deleted files?? I am not sure. Also, build seems to go through without removing first one!!

I'll remove that, build/test again and send another webrev.
One other one is make/tools/src/build/tools/deps/refs.allowed. The last two lines can be replaced with:

java.beans.PropertyChangeListener=java.util.logging.LogManager,compact1,compact2,compact3

and the comment line about Rhino can be removed.

-Alan.

Reply via email to