-
Notifications
You must be signed in to change notification settings - Fork 65
Added graalvm compilation #1443
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
base: master
Are you sure you want to change the base?
Conversation
… a database is used
Added support for windows and linux graalvm compilation
added support for graalvm linux and windows compilation
removed gpg key usage and java install as it's done with graalvm.
please do not commit generated files. Add generated folders into .gitignore. This is too late, so maybe this pull request will be squashed into a single commit to remove commited generated files |
Yes sorry i realized after doing the PR that i did not managed the files generated on a build. I'm doing commits to solve this. |
OK it seems to be good, there's no more generated files in the PR @nicolas-f. |
is the github action generate the native libraries ? |
In both release and snapshot modes, the workflow builds the standalone version, so it should also build the native library and executable (at least for Linux, since the action runs on a Linux runner, not Windows). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the job
First pass with some commentes
h2gis-graalvm/src/main/java/org/h2gis/graalvm/GraalCInterface.java
Outdated
Show resolved
Hide resolved
h2gis-graalvm/src/main/java/org/h2gis/graalvm/GraalCInterface.java
Outdated
Show resolved
Hide resolved
h2gis-graalvm/src/main/java/org/h2gis/graalvm/GraalCInterface.java
Outdated
Show resolved
Hide resolved
h2gis-graalvm/src/main/java/org/h2gis/graalvm/GraalCInterface.java
Outdated
Show resolved
Hide resolved
h2gis-graalvm/src/main/java/org/h2gis/graalvm/GraalCInterface.java
Outdated
Show resolved
Hide resolved
h2gis-graalvm/src/main/java/org/h2gis/graalvm/GraalCInterface.java
Outdated
Show resolved
Hide resolved
h2gis-functions/src/main/java/org/h2gis/functions/factory/H2GISFunctions.java
Outdated
Show resolved
Hide resolved
h2gis-graalvm/src/main/java/org/h2gis/graalvm/GraalCInterface.java
Outdated
Show resolved
Hide resolved
h2gis-graalvm/src/main/java/org/h2gis/graalvm/GraalCInterface.java
Outdated
Show resolved
Hide resolved
h2gis-graalvm/src/main/java/org/h2gis/graalvm/GraalCInterfaceEntryPointFeature.java
Show resolved
Hide resolved
…ectives on linux and windows
With those new code midifications, when building in standalone mode, it create a native lib and executable using graalvm. Linked to issue #1442.