-
Notifications
You must be signed in to change notification settings - Fork 23
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
Upgrade alpine and remove customisation for JNA in the fits image. #215
Comments
Need to wait on this getting in to be able to use php 8: |
@nigelgbanks Do you want to schedule a Zoom call re this task or talk in Slack sometime this week to divvy up next steps between the two of us? |
Linking the existing work here: #238 |
@g7morris I'll ping ya on slack. |
Discussed briefly in the Tech call today; looks like you have a solid plan for this. let us know if you need more community input. |
Even though they added jna back in for aarch64, we can't use it as it's compiled against a newer version of java, so we will have to continue using our own build package. |
Upgraded to JDK 17 (The new LTS for JDK) to get around problems with Fcrepo deadlocking when receiving a SIGTERM. Might work now. |
Yup that worked. |
DOne under #238 |
The following has been merged in
aports
:https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/34087
Which should fix the issue where we had to have customisation to the jna packages for fits to support arm builds.
After upgrading to
3.15.5
when it releases we should be able to cleanup:isle-buildkit/fits/Dockerfile
Lines 60 to 78 in c85af8b
And remove the custom build: https://github.com/Islandora-Devops/isle-buildkit/tree/main/fits/build.
This will require testing on arm to validate that fits is working properly when calling out to native code.
The text was updated successfully, but these errors were encountered: