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

F-Droid build not reproducible #400

Open
linsui opened this issue Sep 10, 2023 · 44 comments
Open

F-Droid build not reproducible #400

linsui opened this issue Sep 10, 2023 · 44 comments
Labels
bug Something isn't working

Comments

@linsui
Copy link

linsui commented Sep 10, 2023

diff.zip

Could you please take a look? Thanks!

@linsui linsui added the bug Something isn't working label Sep 10, 2023
@dessalines
Copy link
Owner

We've had this error before, but I've changed nothing about the signing. All I did was upgrade some dependencies. I'm sure it'll resolve itself like before: in a few releases, f-droid will start working again, and I'll still have changed nothing about the signing config.

@linsui
Copy link
Author

linsui commented Sep 10, 2023

It's not about the signing config. How do you build your apk?

@dessalines
Copy link
Owner

./gradlew assembleRelease

That also hasn't changed, and there's been working and broken f-droid builds before and after that.

@licaon-kter
Copy link
Contributor

@dessalines
Copy link
Owner

What usually causes these?

@linsui
Copy link
Author

linsui commented Sep 21, 2023

Different jdk versions, Android studio, or cache not cleaned. Which jdk do you use to build your apk?

@dessalines
Copy link
Owner

Uh oh, ./gradlew --version says I'm using:

JVM: 17.0.8.1 (Oracle Corporation 17.0.8.1+1)

Yet my java --version is:

openjdk 17.0.8.1 2023-08-24

Maybe these lines are incorrect? https://github.com/dessalines/thumb-key/blob/main/app/build.gradle#L54

@linsui
Copy link
Author

linsui commented Sep 21, 2023

Maybe the oracle jdk produce different binaries. Those lines are correct.

@dessalines
Copy link
Owner

Okay, it looks like doing this is the only true way to figure out what java is being used in the build process.

I'll try setting my JAVA_HOME explicitly to my openjdk version for the next build, and ensuring that it uses it for the process. What a mess.

@licaon-kter
Copy link
Contributor

licaon-kter commented Oct 3, 2023

fyi 2.1.1 not RB either https://gitlab.com/fdroid/fdroiddata/-/commit/2060e300fb57f492478bd3fe94c309a03b63b71b

Difflog is the same as linked above :(

@dessalines
Copy link
Owner

Not sure what I should do to fix it. This one was built with openjdk 17 .

@licaon-kter
Copy link
Contributor

@linsui
Copy link
Author

linsui commented Oct 5, 2023

Hmm, insteresting...

@linsui linsui closed this as completed Oct 5, 2023
@dessalines
Copy link
Owner

I upgraded to gradle 8.3 for that one, not sure what else I did to make that one work.

@licaon-kter
Copy link
Contributor

@dessalines not only gradle, also these c194722

@licaon-kter
Copy link
Contributor

@linsui linsui reopened this Oct 16, 2023
@linsui
Copy link
Author

linsui commented Nov 3, 2023

2.5.2 is reproducible.

@linsui linsui closed this as completed Nov 3, 2023
@dessalines
Copy link
Owner

Might have been the upgrade to gradle 8.4 , but I have no idea lol. I apologize for these.

@licaon-kter
Copy link
Contributor

licaon-kter commented Nov 10, 2023

reopen https://monitor.f-droid.org/builds/log/com.dessalines.thumbkey/43#site-footer

will rebuild locally to see the diff

@linsui linsui reopened this Nov 10, 2023
@licaon-kter
Copy link
Contributor

Difflog:
thumb43.log

Local APK:
com.dessalines.thumbkey_43.apk.ZIP (remove .ZIP)

@linsui
Copy link
Author

linsui commented Jan 4, 2024

2.6.5 failed again.
diff.zip

@dessalines
Copy link
Owner

No idea what causes these. This one is even stranger since I update zero gradle deps: 2.6.4...2.6.5#diff-51a0b488f963eb0be6c6599bf5df497313877cf5bdff3950807373912ac1cdc9

@licaon-kter
Copy link
Contributor

@licaon-kter
Copy link
Contributor

at least it's a new error https://gitlab.com/fdroid/fdroiddata/-/jobs/6133897492#L566

@licaon-kter
Copy link
Contributor

https://gitlab.com/fdroid/fdroiddata/-/jobs/6145358484#L600

Only in /tmp/tmp2thsd9qi/tmp_binaries_com.dessalines.thumbkey_57.binary/content/assets: RELEASES.md

@linsui
Copy link
Author

linsui commented Feb 12, 2024

@dessalines
Copy link
Owner

dessalines commented Feb 12, 2024

I spose I just need to make sure that file is added to git then. I'll try that for the next release.

@dessalines
Copy link
Owner

Why did 2.6.15 fail? I added the missing file for that one.

@licaon-kter
Copy link
Contributor

difflog:
thumb58.log

@dessalines
Copy link
Owner

I'm not sure how to read these, but it says that even things like ArrayList differ.

@linsui
Copy link
Author

linsui commented Feb 18, 2024

@quasipedia
Copy link

Is there anything that somebody with zero knowledge of Android coding can do to help here? The version on F-droid is lagging behind of more than 10 releases, and some critical bugs (like missing of letters from some layout) are still plaguing F-droid users... 😿

@licaon-kter
Copy link
Contributor

I keep building newer versions locally, no success yet.

will rebuild latest asap

@licaon-kter
Copy link
Contributor

new difflog:
thum65.log

@licaon-kter
Copy link
Contributor

@dessalines
Copy link
Owner

Nice. Again I wish I knew what causes these.

@licaon-kter
Copy link
Contributor

🤷 https://gitlab.com/fdroid/fdroiddata/-/commit/8fb6877c1ccca1b1756601972b9674d17a6d243a

@dessalines
Copy link
Owner

It fixed itself and we have no idea why lol. I'll take it.

@licaon-kter
Copy link
Contributor

Don't jinx it now :)

@licaon-kter
Copy link
Contributor

and :( again https://gitlab.com/fdroid/fdroiddata/-/jobs/6554341767#L547

@dessalines
Copy link
Owner

Not even a dep update with that one 🤷

@licaon-kter
Copy link
Contributor

I'am just keeping you annoyedinformed :)

@licaon-kter
Copy link
Contributor

licaon-kter commented Apr 24, 2024

@dessalines pls always build APK from Tagged commit, eg. https://github.com/dessalines/thumb-key/releases/tag/3.1.0 -> 1abe5ce

else https://gitlab.com/fdroid/fdroiddata/-/jobs/6702007748#L1257

not to mention that the commit is the one with the right versionCode and Name :(

/LE: fyi https://gitlab.com/fdroid/fdroiddata/-/commit/a5c31f0211a2e46bff69c8b311aff68a614aca08

@dessalines
Copy link
Owner

dessalines commented Apr 24, 2024

Oops my bad. I'll reupload another one shortly.

Also:

git rev-list -n 1 3.1.0
1abe5ce59dfb9cd52aea5a5c5575259067efbee6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants