RFR: 8305858: Resolve multiple definition of 'handleSocketError' when statically linking with JDK native libraries

Wed Apr 12 03:29:37 UTC 2023

On Wed, 12 Apr 2023 03:20:47 GMT, Jiangli Zhou <jiangli at openjdk.org> wrote:

> Rename 'handleSocketError' to 'sctpHandleSocketError' in libsctp. This resolves the related duplicate symbol failures when both libnio and libsctp are statically linked with the launcher.
> 
> Remove #include "nio_util.h" from SctpChannelImpl.c and SctpNet.c. Looks like that's only needed for handleSocketError() declaration. Declare sctpHandleSocketError() in Sctp.h.
> 
> (Sorry about the earlier RFR/Withdrawn noise.)

I noticed there is a presubmit test failure: https://github.com/jianglizhou/jdk/actions/runs/4672962155#user-content-java_lang_thread_virtual_tracevirtualthreadlocals for linux-x86. It appears to be unrelated. Will file a bug tomorrow if there is not existing bug (will check before filing).

-------------

PR Comment: https://git.openjdk.org/jdk/pull/13438#issuecomment-1504522861