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

Fix compilation for the newer Java envs #1539

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

yunikkk
Copy link

@yunikkk yunikkk commented Feb 27, 2023

Added couple workarounds to solve the issues reported by a customer:

tested make build-cli but got this error :
FAILURE: Build failed with an exception.

  • Where:
    Script 'java/mapbox/mapbox-java/gradle/artifact-settings.gradle' line: 32

  • What went wrong:
    A problem occurred evaluating script.
    String index out of range: -1
    then in getVersionName in artifact-settings.gradle, I overrided the version from what git describe --tag --abbrev=0 gives me (v6.11.0-beta.1) to "1.0.0" , but then make build-cli fails with :
    java/mapbox/mapbox-java/services-core/build/gen/buildconfig/src/main/com/mapbox/core/BuildConfig.java:4: error: cannot access Object
    public final class BuildConfig
    ^
    bad class file: /modules/java.base/java/lang/Object.class
    class file has wrong version 55.0, should be 53.0
    Please remove or make sure it appears in the correct subdirectory of the classpath.
    1 error

It seems with recent JVM (e.g. mine is openjdk 11.0.17 2022-10-18 LTS) builds are broken at the moment.
Haven't found compatible net.ltgt.errorprone settings, tried couple different verisions with no luck. Could someone familiar with the repo from pick it up cc @mapbox/navigation-android?

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.

1 participant