jabstone / JABtalk

JABtalk Android App
GNU General Public License v2.0
11 stars 9 forks source link

gradle-wrapper.jar outdated #10

Closed IzzySoft closed 2 months ago

IzzySoft commented 2 months ago

Your /gradle/wrapper/gradle-wrapper.jar is 9 years old and no longer matching current gradle versions. While you just updated the gradle-wrapper.properties, you probably forgot to upgrade the JAR as well. Could you please do so? Thanks in advance!

If you wonder about the background: JABtalk is listed at IzzyOnDroid. At IzzyOnDroid we support Reproducible Builds (see: Reproducible Builds, special client support and more at IzzyOnDroid). JABtalk succeeded as such (congrats!) and thus its listing shows the "green shield" confirming this. Currently, whenever you change the gradle version, the recipe at the verification builder requires a manual update as your JAR is not recognized – which leads to delays of confirmation, or to a release being marked "RB failed". An up-to-date gradle-wrapper.jar easily avoids that :wink: So thanks for your support!

jabstone commented 2 months ago

I just updated it and pushed the changes. If it still doesn't work, I'll need more info on how to upgrade the file correctly.

IzzySoft commented 2 months ago

That was fast, thanks! I've checked but funnily still get

+ /opt/gradle-wrapper-verify/gradle-wrapper-verify ./gradle/wrapper/gradle-wrapper.jar
checking ./gradle/wrapper/gradle-wrapper.jar ...
unknown hash

even when building from the latest commit (80aeb41726858201071ca511e551cd3a7a64a913 – which replaced the JAR). Where did you take the JAR from?

cc @obfusk (who wrote the gradle-wrapper-verify tool)

jabstone commented 2 months ago

I regenerated it via ./gradlew wrapper --gradle-version 8.10.1. I followed these directions:

`Replace the gradle-wrapper.jar To ensure your gradle-wrapper.jar file is up-to-date, you can regenerate it using the Gradle Wrapper command.

Here are the steps:

Open a terminal in your project’s root directory. Run the following command to update the wrapper JAR and other related files:

./gradlew wrapper --gradle-version [VERSION] Replace [VERSION] with the latest Gradle version (e.g., 8.10.1).`

obfusk commented 2 months ago

@IzzySoft the issue seems to be on your end:

$ git clone https://github.com/jabstone/JABtalk
$ cd JABtalk
$ git checkout 80aeb41726858201071ca511e551cd3a7a64a913
$ gradle-wrapper-verify
checking gradle/wrapper/gradle-wrapper.jar ...
OK
IzzySoft commented 2 months ago

@obfusk funny, that's the recipe I've used:

  - tag: 5.3.2
    apks:
      - apk_pattern: jabtalk.*\.apk
        apk_url: https://github.com/jabstone/JABtalk/releases/download/5.3.2/jabtalk_v5.3.2.apk
        build:
          - git checkout 80aeb41726858201071ca511e551cd3a7a64a913
          - ./gradlew assembleRelease
          - mv app/build/outputs/apk/release/*unsigned.apk /outputs/unsigned.apk

Let me try again… Nope, fails. Looking at the logs I miss output from git checkout. Eek: of course gradle is processed before the build recipe is started with – so the checkout in the recipe comes too late :man_facepalming: Let me try with com.jabstone.jabtalk.basic:5.3.2:80aeb41726858201071ca511e551cd3a7a64a913:none

:bow:

PEBKAC. Apologies, @jabstone – all seems fine now! This time the error indeed was on my end. Should have thought of that…