[8u40] approval request for 8044146: jdk.nashorn.api.scripting package javadoc should be included in jdk docs

Discussion:

A. Sundararajan

2014-12-01 12:02:19 UTC

Please approve

bug: https://bugs.openjdk.java.net/browse/JDK-8066146
8u-dev webrev: http://cr.openjdk.java.net/~sundar/8066146/8u40/webrev.00/
jdk9 review thread:
http://mail.openjdk.java.net/pipermail/nashorn-dev/2014-November/003922.html

I had to adjust paths in makefile slightly for jdk8u40. CC'ing jdk9
reviewers as well...

Thanks,
-Sundar

Marcus Lagergren

2014-12-01 12:18:45 UTC

Permalink

Post by A. Sundararajan
Please approve
bug: https://bugs.openjdk.java.net/browse/JDK-8066146
8u-dev webrev: http://cr.openjdk.java.net/~sundar/8066146/8u40/webrev.00/
jdk9 review thread: http://mail.openjdk.java.net/pipermail/nashorn-dev/2014-November/003922.html
I had to adjust paths in makefile slightly for jdk8u40. CC'ing jdk9 reviewers as well...
Thanks,
-Sundar

Jim Laskey (Oracle)

2014-12-01 12:54:20 UTC

Permalink

Hannes Wallnoefer

2014-12-01 12:44:30 UTC

Permalink

Post by A. Sundararajan
Please approve
bug: https://bugs.openjdk.java.net/browse/JDK-8066146
8u-dev webrev: http://cr.openjdk.java.net/~sundar/8066146/8u40/webrev.00/
http://mail.openjdk.java.net/pipermail/nashorn-dev/2014-November/003922.html
I had to adjust paths in makefile slightly for jdk8u40. CC'ing jdk9
reviewers as well...
Thanks,
-Sundar

Rob McKenna

2014-12-01 15:51:51 UTC

Permalink

Approved. Please add a suitable noreg keyword.

-Rob

Post by A. Sundararajan
Please approve
bug: https://bugs.openjdk.java.net/browse/JDK-8066146
8u-dev webrev: http://cr.openjdk.java.net/~sundar/8066146/8u40/webrev.00/
http://mail.openjdk.java.net/pipermail/nashorn-dev/2014-November/003922.html
I had to adjust paths in makefile slightly for jdk8u40. CC'ing jdk9
reviewers as well...
Thanks,
-Sundar