On 9.12.2015 14:55, Severin Gehwolf wrote:
On Tue, 2015-12-01 at 14:19 +0100, Severin Gehwolf wrote:
Hi Jaroslav,
On Tue, 2015-12-01 at 12:33 +0100, Jaroslav Bachorik wrote:
On 1.12.2015 11:17, Severin Gehwolf wrote:
On Mon, 2015-11-09 at 10:32 +0100, Severin Gehwolf wrote:
On Wed, 2015-11-04 at 11:54 +0100, Severin Gehwolf wrote:
Hi,
Updated webrev with jtreg test in Java:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6425769/02/
bug: https://bugs.openjdk.java.net/browse/JDK-6425769
I believe this updated webrev addresses all concerns and
incorporates
suggestions brought up by Jaroslav and Daniel.
I'm still looking for a sponsor and a hotspot/servicability-
dev
reviewer. Could somebody maintaining javax.rmi.ssl have a
look at
this
as well? Thank you!
Ping? Friendly reminder that I still need reviewers and a
sponsor for
this.
Anyone?
I'm sorry for not spotting this earlier:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6425769/03.no-rmi-
ssl-factory-
changes/jdk/src/java.management/share/classes/sun/management/jmxrem
ote/ConnectorBootstrap.java.sdiff.html
* L442 - the log would contain 'com.sun.management.jmxremote.host
=
null' if host is not specified; might be better not to print this
out at all
Updated webrev which does not print
'com.sun.management.jmxremote.host = null' if unset:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6425769/05/
Other than that the change looks good to me. If no one else is
volunteering I may sponsor this change.
Thank you!
Jaroslav, are you still willing to sponsor this? There hasn't been much
movement :-/
Sure. I need to start the compatibility review process before
integration and it might take some time.
-JB-
Thanks,
Severin