Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Netty OpenSsl.isAvailable() check doesn't handle lib loading issue on 32bit JVM #307

Closed
doom369 opened this issue Sep 27, 2017 · 1 comment
Closed

Netty OpenSsl.isAvailable() check doesn't handle lib loading issue on 32bit JVM #307

doom369 opened this issue Sep 27, 2017 · 1 comment
Assignees
@normanmaurer
Milestone
2.0.7.Final

Comments

@doom369
Copy link

doom369 commented Sep 27, 2017 •
edited
Loading

Details are here : netty/netty#7150 (comment)

Brief summary:

Right now tcnative library has name libnetty_tcnative.so and this name doesn't include OS classifier that prevents netty from loading this library if system architecture doesn't match with packed version. For example, epoll native library name look like this libnetty_transport_native_epoll_x86_64.so.
@normanmaurer normanmaurer self-assigned this Oct 24, 2017
normanmaurer added a commit that referenced this issue Oct 24, 2017
@normanmaurer
Include architecture in native lib name.
c05417c 
Motivation:

We should include the architecture in the native lib name to allow not load it if the architecture not match.
This is related to netty/netty#7163.

Modifications:

Add architecture to native lib name.

Result:

Fixes [#307].
@normanmaurer normanmaurer mentioned this issue Oct 24, 2017
Closed
@normanmaurer
Copy link
Member

@doom369 PTAL #315

normanmaurer added a commit that referenced this issue Oct 24, 2017
@normanmaurer
Include architecture in native lib name.
f49f7de 
Motivation:

We should include the architecture in the native lib name to allow not load it if the architecture not match.
This is related to netty/netty#7163.

Modifications:

Add architecture to native lib name.

Result:

Fixes [#307].
normanmaurer added a commit that referenced this issue Oct 24, 2017
@normanmaurer
Include architecture in native lib name.
46cbbb4 
Motivation:

We should include the architecture in the native lib name to allow not load it if the architecture not match.
This is related to netty/netty#7163.

Modifications:

Add architecture to native lib name.

Result:

Fixes [#307].
normanmaurer added a commit that referenced this issue Nov 1, 2017
@normanmaurer
Include architecture in native lib name.
1eeb4e1 
Motivation:

We should include the architecture in the native lib name to allow not load it if the architecture not match.
This is related to netty/netty#7163.

Modifications:

Add architecture to native lib name.

Result:

Fixes [#307].
@normanmaurer normanmaurer added this to the 2.0.7.Final milestone May 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@normanmaurer normanmaurer

Labels
None yet
Projects
None yet
Milestone
2.0.7.Final
Development

No branches or pull requests
2 participants
@normanmaurer @doom369