The change without ORB.java looks okay to me.
Mandy
On 2/10/15 4:11 AM, Mark Sheppard wrote:
OK I'll remove it.
I thought that property files had been migrated from lib to conf, as
per conf/security, so I made the change
regards
Mark
On 10/02/2015 11:37, Alan Bateman wrote:
On 10/02/2015
On 07/02/2015 00:22, Mark Sheppard wrote:
Hi Alan,
I had meant to remove the commented lines prior to generating the
patch
Okay, so ignoring that part then the rest looks good to me. Hopefully we
have enough tests in this area that run with a security manager to help
find any issues.
On 10/02/2015 11:20, Mark Sheppard wrote:
thanks Alan
the updated corba part is at
http://cr.openjdk.java.net/~msheppar/8068682/corba/webrev.02/
I assume ORB.java isn't meant to be in this webrev (the lib-conf issue
is separate and I think will need an @implNote in additional to checking
thanks Alan
the updated corba part is at
http://cr.openjdk.java.net/~msheppar/8068682/corba/webrev.02/
regards
Mark
On 10/02/2015 09:14, Alan Bateman wrote:
On 07/02/2015 00:22, Mark Sheppard wrote:
Hi Alan,
I had meant to remove the commented lines prior to generating the
patch
Okay,
OK I'll remove it.
I thought that property files had been migrated from lib to conf, as per
conf/security, so I made the change
regards
Mark
On 10/02/2015 11:37, Alan Bateman wrote:
On 10/02/2015 11:20, Mark Sheppard wrote:
thanks Alan
the updated corba part is at
On 08/02/2015 06:58, Mandy Chung wrote:
:
Mark's patch is first to move it out from the boot loader to the ext
class loader. Identifying the permissions required by corba requires
more effort and not straight-forward. So it's granted with
AllPermissions for now and reduce the permission
On 7/02/2015 4:55 AM, Mark Sheppard wrote:
Hi
please oblige and review the following changes
http://cr.openjdk.java.net/~msheppar/8068682/webrev/
http://cr.openjdk.java.net/~msheppar/8068682/corba/webrev/
which address the issue in
https://bugs.openjdk.java.net/browse/JDK-8068682
this
On 2/7/2015 5:10 PM, David Holmes wrote:
On 7/02/2015 4:55 AM, Mark Sheppard wrote:
Hi
please oblige and review the following changes
http://cr.openjdk.java.net/~msheppar/8068682/webrev/
http://cr.openjdk.java.net/~msheppar/8068682/corba/webrev/
which address the issue in
On 06/02/2015 18:55, Mark Sheppard wrote:
Hi
please oblige and review the following changes
http://cr.openjdk.java.net/~msheppar/8068682/webrev/
http://cr.openjdk.java.net/~msheppar/8068682/corba/webrev/
which address the issue in
https://bugs.openjdk.java.net/browse/JDK-8068682
this change
Hi Alan,
I had meant to remove the commented lines prior to generating the patch
regards
Mark
On 06/02/2015 21:21, Alan Bateman wrote:
On 06/02/2015 18:55, Mark Sheppard wrote:
Hi
please oblige and review the following changes
http://cr.openjdk.java.net/~msheppar/8068682/webrev/
Hi
please oblige and review the following changes
http://cr.openjdk.java.net/~msheppar/8068682/webrev/
http://cr.openjdk.java.net/~msheppar/8068682/corba/webrev/
which address the issue in
https://bugs.openjdk.java.net/browse/JDK-8068682
this change means CORBA ORB is loaded by the extension
11 matches
Mail list logo