Please review changes for JDK-8012975: Remove rhino from jdk8

Alan Bateman Alan.Bateman at oracle.com
Fri May 3 09:26:09 UTC 2013


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.



More information about the build-dev mailing list