Re: Reqest for Review: Fix bug in FDS on Solaris

2012-06-05 Thread Magnus Ihse Bursie
On 2012-06-05 03:32, Kelly O'Hair wrote: I'm waiting on Dan, and I'm doing a dryrun through JPRT. Was there a CR on this? No, sorry. And should I really be integrating this FDS change, or should Dan? I don't know. I don't have any opinion on that. If it comes into the build forest directly

Re: Review Request: Build-infra update

2012-06-05 Thread Alan Bateman
On 04/06/2012 15:15, Erik Joelsson wrote: I have created a (hopefully temporary) hack to run javah manually for these 5 classes. This webrev is just against the build-infra repo. Unless anybody objects to this temporary solution, I will publish a new full webrev against the jdk8/build forest

Re: Review Request: Build-infra update

2012-06-05 Thread Alan Bateman
On 04/06/2012 17:52, Kelly O'Hair wrote: ./share/classes/sun/nio/ch/sctp/SctpStdSocketOption.java:@GenerateNativeHeader ./solaris/classes/sun/nio/ch/sctp/AssociationChange.java:@GenerateNativeHeader ./solaris/classes/sun/nio/ch/sctp/PeerAddrChange.java:@GenerateNativeHeader

Re: Reqest for Review: Fix bug in FDS on Solaris

2012-06-05 Thread Daniel D. Daugherty
On 6/4/12 9:00 AM, Magnus Ihse Bursie wrote: As previously discussed here, there is a bug basically disabling FDS on Solaris. Here is a webrev for a patch from Jonas Oreland (jonas.orel...@oracle.com; he's not on this list though).