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.

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

Jdk10/issue12 #140

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Jdk10/issue12 #140

wants to merge 2 commits into from

Conversation

0x6e3078
Copy link

tldr: Remove the usage of javah to support JDK above 8.

With the JDK8 the javah command is deprecated and is removed in jdk9, the function is now supported by the javac command. But the source code needs the java native annotation. The Patch add the necessary annotations and enable the header generation by the compiler.

with the jdk8 the javah command is deprecated and is removed at least
in the jdk10. The javac is now doing the work, the necessary changes
are added to the pom.xml.

To mark the parts which should be exposed into the header files, it is
important to use the java native annotation @Native.

Only the files which contains exposed information are generated.
@codecov-io
Copy link

Codecov Report

Merging #140 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #140   +/-   ##
========================================
  Coverage      46.4%   46.4%           
  Complexity      892     892           
========================================
  Files            60      60           
  Lines          9875    9875           
  Branches       1611    1611           
========================================
  Hits           4582    4582           
  Misses         4830    4830           
  Partials        463     463

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants