File a separate to track jdk.dynalink permissions:
https://bugs.openjdk.java.net/browse/JDK-8145868
Adjusted test comments that Hannes mentioned and pushing changes in
webrev @ http://cr.openjdk.java.net/~sundar/8145750/webrev.01/
Thanks
-Sundar
On 12/20/2015 9:53 PM, Alan Bateman wrote:
On 18/12/2015 12:55, Sundararajan Athijegannathan wrote:
May be, not. But I tried giving only sun.reflect package access --
didn't work. There are a few doPrivileged blocks in dynalink code as
well. This needs further analysis.
But dynalink code was part of nashorn and so was getting
AllPermission so far - so there is no permission enhancement by
adding this missing permission block.
That said, we can revisit reduced permission set for dynalink module.
I'd prefer to track that as separate bug.
Sure, a separate bug is fine.
If test/tools/jjs is new then it would be good to this test
directory into one of the test groups so that the tests are run,
maybe jdk_other.
okay, I'll find out that.
Looks looks core_tools will execute tools/** so I think what you have
here is okay.
-Alan.
