Skip to content

Conversation

kotogadekiru
Copy link

Note: Filling out this template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the
maintainer's discretion.

Description of the Change

make it posible to compile with java 17 and update to jogl 2.6 and jackson 2.17

Why Should This Be In Core?

Benefits

make use of the new jvm facilities like jlink and jpackage, new garbage collectors, better native management, records,

Potential Drawbacks

users will need to upgrade their jvm version

Applicable Issues

@EMaksymenko
Copy link
Member

Why do not you add gdal via maven dependency instead of jar?

@kotogadekiru
Copy link
Author

i just followed the instructions to update gdal.
this project is built by gradle not maven.
maybe there's a limitation in the repository as gdal is not on central.

@EMaksymenko
Copy link
Member

Gdal is available on central.
Try to use dependencies from Maven Central repository.

@kotogadekiru
Copy link
Author

there seems to be two different conflicting set of instructions on how to upgrade gdal.
on README_DEVELOPER.TXT states:

Updating the JOGL Library

  1. Download the JOGL deployment package archive/jogamp-all-platforms.7z for the new JOGL version. JOGL deployment
    packages are organized by version at the following URL:
    ###http://jogamp.org/deployment/
  1. Extract the archive, then copy the following 15 files to the WorldWind Java project root:
    gluegen-rt-natives-linux-amd64.jar
    gluegen-rt-natives-linux-i586.jar
    gluegen-rt-natives-macosx-universal.jar
    gluegen-rt-natives-windows-amd64.jar
    gluegen-rt-natives-windows-i586.jar
    gluegen-rt.jar
    gluegen.LICENSE.txt
    jogl-all-natives-linux-amd64.jar
    jogl-all-natives-linux-i586.jar
    jogl-all-natives-macosx-universal.jar
    jogl-all-natives-windows-amd64.jar
    jogl-all-natives-windows-i586.jar
    jogl-all.jar
    jogl.LICENSE.txt
    jogl.README.txt
  1. Update the JOGL version in the following files:
    README.txt
    README_DEVELOPER.txt (this document)
    release-build.xml (JOGL link in javadocs target)

this is what i did and it works.
but on GDAL_README.txt states:

When using the Gradle build script, note that the version number
defined by 'gdalVersion' must match the binary libraries that are
installed. The Gradle build script pulls the 'gdal.jar' from
Maven or JCenter. If you want to use 'gdal.jar' from the local
disk, change the Gradle script to specify

     dependencies {
         ...
          compile files('gdal.jar')
          ...
      }

am i mixing gdal with jogl and shouldn't?
that's probably it.
will try.

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.

3 participants