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

GDAL base docker image #535

Closed
wants to merge 2 commits into from
Closed

Conversation

jemacchi
Copy link
Collaborator

@jemacchi jemacchi commented Oct 1, 2024

GDAL related base image.
Builds GDAL jar file based on eclipse-temurin:11-jdk (noble release, java sdk 11)
Then copy so+jar files in tmp folder for next multistage builds

@jemacchi jemacchi requested a review from groldan October 1, 2024 12:18
src/apps/base-images/gdal/Dockerfile Show resolved Hide resolved
ENV GDAL_DATA $GDAL_PATH/3.8.4
ENV PATH $GDAL_PATH:$PATH
ENV LD_LIBRARY_PATH $LD_LIBRARY_PATH:/usr/lib/jni:/usr/share/java:/usr/local/lib/:/usr/lib/
ENV ANT_VERSION 1.9.16
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this environment variable actually needed? If so, it might get out of sync with the required ant version since it might change in the package repository.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeap, ANT_VERSION is not required (removing it)

src/apps/base-images/gdal/Dockerfile Outdated Show resolved Hide resolved
Update src/apps/base-images/gdal/Dockerfile

Co-authored-by: Daniel Koch <[email protected]>
This was referenced Oct 1, 2024
@jemacchi jemacchi closed this Oct 1, 2024
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.

4 participants