-
-
Notifications
You must be signed in to change notification settings - Fork 459
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
APK files missing at releases sinve v3.8.1 #4053
Comments
I don't understand the big report, all APK are here https://download.kiwix.org/release/kiwix-android/ |
@IzzySoft We have updated the our release |
@MohitMaliFtechiz Can you easily do that? Do we know why this is not there anymore? |
Yes, I was checking why it was not there and I found, in those releases which we self made these APKs are not available because these APKs are uploaded by our kiwix-android/.github/workflows/release.yml Lines 49 to 55 in 1576095
It is not occurring after the release |
It needed the manual steps to do it:
So to reduce the manual steps, we have now directly uploaded the APKs to the existing release instead of creating the new draft release in #4054. |
@kelson42 the updater at the IzzyOnDroid repo looks for them at the releases of the repo hosting the source code. @MohitMaliFtechiz thanks for fixing! 🤩 Funnily the updater here now picks 3.8.1 as latest one (most likely because the Github API returns the releases by date, and 3.8.1 has the newest timestamp), but that should "fix itself" with the next release then hopefully:
Oops, some exclamation marks 😱 Let's see… I cannot find details on permissions in the repo (or in its wiki), so:
As for android {
dependenciesInfo {
// Disables dependency metadata when building APKs.
includeInApk = false
// Disables dependency metadata when building Android App Bundles.
includeInBundle = false
}
} For some background: that BLOB is supposed to be just a binary representation of your app's dependency tree. But as it's encrypted with a public key belonging to Google, only Google can read it – and nobody else can even verify what it really contains. More details can be found e.g. here: Ramping up security: additional APK checks are in place with the IzzyOnDroid repo. Thanks in advance! |
We have a mess here (also to many workflow files). We have to clean and republish the APK (but manual release triggering). I will have to look |
The
@IzzySoft We require the
For this, we have opened a ticket #4055.
Thanks for this, we will fix this.
@IzzySoft, @kelson42 It was published by mistake by github action while fixing and testing this issue. My bad 😥
Thanks for doing this ❤️ |
Thanks @MohitMaliFtechiz 😍 Permissions are added to Kiwix' "green list" now: android.permission.ACCESS_COARSE_LOCATION: used on Android < 13 to discover peer-to-peer (P2P) devices for transferring ZIM files to nearby devices
android.permission.ACCESS_FINE_LOCATION: used on Android < 13 to discover peer-to-peer (P2P) devices for transferring ZIM files to nearby devices
android.permission.NEARBY_WIFI_DEVICES: used on Android ≥ 13 to discover peer-to-peer (P2P) devices for transferring ZIM files to nearby devices
android.permission.MANAGE_EXTERNAL_STORAGE: used to load the sideloaded ZIM files on Android ≥ 11
android.permission.READ_EXTERNAL_STORAGE: used to load the sideloaded ZIM files on Android < 11 Already visible if you visit Kiwix in the IzzyOnDroid repo browser. Please correct if I abbreviated/deduced wrongly. Thanks for splitting off the cleartextTraffic to follow it up – and for taking care for the
No worries, it's fixed here by now. And as I wrote, the next release will "close the gap" again.
Gladly – thanks for your support and swift action! |
Thanks again! Btw, the F-Droid badge on your Readme links to a 404. Maybe you want to pick an IzzyOnDroid badge and link it to your app there instead, @MohitMaliFtechiz? |
|
Thanks! |
@IzzySoft We have just released the version Edited |
Triggering manually what would have run automatically later today:
Looks good so far. But, we don't have that app:
What we have here is
(and btw, there's no direct update path from
No, the updater here checks the releases for files matching the defined pattern, For Kiwix, that pattern is |
@IzzySoft Thank you for your response.
It was due to #3933 and I have opened a separate ticket for this problem #4114. |
@IzzySoft We have updated our release with
@IzzySoft Why it is only taking the So either it should upload all abi APKs on IzzyOnDroid or should take the universal APK from github release so that it can be installed on different architecture devices. Like it is uploading different abi APKs for the |
Thanks! That will then be picked up with the next run of the updater later today.
We have a size limit of 30 MB per app, so we had to decide for one. Back then it was decided for armeabi because it offers the widest compatibilities (excluding only very few devices like the Pixel 7a plus, where vendors decided to make the OS "arm64 only". At IzzyOnDroid, we want to support longevity, to enable the use of older devices (e.g. with custom ROMs) for as long as they last (I e.g. still use a 2015 Fairphone 2, which is 32bit and thus cannot support 64bit apps).
Multiple APKs per release are not supported (even at F-Droid they'd then need a unique
That's a single APK covering all ABIs ("universal APK"). You've cut off the screenshot before the APK size, but that APK there is much smaller: just 2 MB per APK. |
It is already there in our code, all the APKs are generated with unique versionCode.
Ohh!! I was not aware of that, Thanks for the explanation. Now I can understand why only |
With 32bit devices slowly aging out (and in case of Kiwix, I'm not even sure if they're still capable of running the app with an acceptable "fluidity"), we can of course switch to the arm64 build later. Btw, as you might be aware of, it's the native libs which per APK make up ~18 MB, and are stored uncompressed. I've no idea if any size optimization is possible there, but you've probably checked that already. |
Yes, We have done the all optimization to minimize the APK size. |
Could you please add the APK files again to your releases (starting with the current one would be fine, no need to fill up the older ones)? Else Kiwix cannot be kept updated at e.g. IzzyOnDroid (which currently is stuck at 3.8.0 due to this).
Thanks in advance!
The text was updated successfully, but these errors were encountered: