Suggestion of combining some macros of processing solaris, macosx with other UNIX

Frank Ding dingxmin at linux.vnet.ibm.com
Thu Aug 16 02:21:06 PDT 2012


On 8/14/2012 10:42 PM, Chris Hegarty wrote:
> On 14/08/12 13:11, Alan Bateman wrote:
>> On 14/08/2012 04:11, Frank Ding wrote:
>>> On 8/7/2012 1:46 PM, Frank Ding wrote:
>>>> :
>>>>
>>>> Could anybody take a look at my patch below and make comment?
>>>> http://cr.openjdk.java.net/~youdwei/ojdk-533/webrev.00/
>>>>
>>>> Thanks & Best regards,
>>>> Frank
>>>
>>> Hi all,
>>>   Is there anybody who is interested in the patch and who can take a
>>> look and comment?
>> It looks okay to me but I don't have time at the moment to sponsor it.
>> Can you confirm that you've run the tests with this change?
>
> I filed 7191275: "Cleanup OS specific blocks in 
> PlainDatagramSocketImpl.c to support more unix-like platforms",for 
> this issue.
>
> I can sponsor this patch and help get it in. Can answer Alan's 
> question about testing? And confirm that it builds on all platforms?
>
> Thanks,
> -Chris.
>
>>
>> -Alan
>
Hi Chris and Alan,
   Thank you for taking time to help this issue.  I have built using 
latest openjdk 8 repo on Windows 64 and Linux 32/64.  Since it's a macro 
change in path "src/solaris", I only did jtreg tests for Linux 32 and 64 
build.  The jtreg tests I ran are restricted to package "java/net".  
Please let me know if you need me to do more tests or on more platforms 
(such as Solaris).

Best regards,
Frank




More information about the net-dev mailing list