-
Notifications
You must be signed in to change notification settings - Fork 7
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
Fastlane issues #96
Comments
Hi @IzzySoft Thanks for your interest in OpenAthena! Theta's business is predicated on the successful delivery of trustable, signed, open source software products. Availability on F-Droid and 3rd party repositories like yours is a high priority for our firm. I haven't been keeping the I believe F-Droid keys off of GitHub version tags, so I will make a minor version release once the metadata is updated. Please let me know if you have any issues with this repo thereafter. I can make further edits and even review pull requests to make sure our repo is usable. You are free to distribute OpenAthena for Android as you wish, on my end however I will need to review IzzySoft's method of distribution before adding a badge to the README.md. We only link to distributions which provide binaries signed with my private key. If you have any other questions or concerns, feel free to reach out to me at [email protected] -Matthew |
Hi @mkrupczak3 and thanks for your response! Yes, I indeed noticed that in the logs of my updater (my repo, too, pulls fastlane from here; other than F-Droid it doesn't pull directly at tag level but picks up what is provided at
Sad to read this! Did it get stuck, or was it closed/rejected? I don't see any show-stoppers in my scanner's report, but F-Droid runs additional checks and has harder requirements than my repo.
Then you'll probably also love to add the above mentioned badge to accompany the currently lonely PlayStore badge 😃
So go right ahead! My updater picks the APKs you provide at your releases here. And for security reasons makes sure the signing key wasn't changed (e.g. by some malicious actor having somehow gained access to an app's repo), so you can check your certificate and compare its SHA256 to the one the app is pinned to here:
That indeed would automatically trigger an update here within 24h. Other than F-Droid, I can also trigger a manual reload of fastlane data separately – but an update would bring the app "to the front page" again causing more people to take a look and see your refreshed descriptions, screenshots etc.
Thanks, I'll gladly take another look then – and if you wish even a closer one to be able to give some hints on possible improvements. Until then, be welcome to take a look at: As for reviewing the process: you can find most details (including the code used) at the corresponding GitLab repo – and feel free to ask if something was left unclear. TL;DR: APKs are pulled from releases, metadata from Fastlane if they exist. APKs are checked with my library scanner (which is also used by F-Droid) and against VirusTotal, with the results being made transparent to users on the WebIF (there's unfortunately no way yet to inject it to the data shown by the F-Droid Android clients). The updater usually runs once a day for the regular apps; for apps which are rarely updated (e.g. haven't seen a commit for a year) there's a separate monthly run. |
Ok, I've updated the en-US changelogs in 3de9b1b and added localized descriptions for each language. I need to add some screenshots in a few languages, then I will push out a tag v0.19.2 I'll attempt to get a submission to F-Droid shortly after. Will add the IzzySoft badge in first commit after v0.19.2 tag |
Out of curiosity, how did you hear about OpenAthena for Android? We've only recently started marketing efforts, feedback could be helpful |
@IzzySoft I updated metadata for all locales in 8e6efce A new release, v0.19.2 with version code 27 is now live: I was able to build the app using the The APKSigningBlock and APKSigningBlockOffset files for reproducible builds are available from the MR here: It should be ready for inclusion with IzzySoft given these changes. Let me know if things are working on your end. I'll add the IzzySoft badge to this repo soon. |
The app contains two (non-code) binary data files:
...which cause the default FDroid build pipeline to fail (due to it misinterpreting them as non-free code binaries). The project is buildable with the |
One of my crawlers found it back in February 😉
"Ready for update" you mean 😝 OK, let's go:
(and 2 more pages with the fastlane details) – thanks, all pulled fine, and no errors 😃 Update will go live with the next sync around 6 pm UTC. Btw: you can use "simple HTML" in your Please see <a href='https://OpenAthena.com/'>OpenAthena.com</a> for more information (thanks for including changelogs with fastlane – they are now integrated as well as you can see)
Ah, I see. So that tripped the scanner and resulted in an error. As it's easy to explain (and not a real issue), those files can be "whitelisted" in the build recipe at F-Droid using |
Great, thanks! I see that your inclusion with F-Droid.org is progressing well, congrats! And it seems you even established reproducible builds (though at the moment it's not clear whether that succeeded, as you used the method that requires you to submit the signing blocks per MR – which also means that IMHO Auto-Update cannot work as designed), cool! So once your app shows up there, do you want me to keep it in my repo? As by that method, updates to F-Droid would (if I understand correctly) always require manual work: a MR with the signing blocks that must be approved and merged, so updates will take significantly longer. I wonder why you've chosen that path instead of using Binaries: https://github.com/Theta-Limited/OpenAthenaAndroid/releases/download/%v/OpenAthena_for_Android_%v.apk which could be handled automatically ( |
Hi, Thanks for the feedback, It's been very helpful for my submission process to FDroid I'm going to implement your suggestion to use the Fixing up another commit for that MR now... |
I've added a couple of new commits to fix syntax and implement the Watching the build process now |
Reproducible builds seem to be failing, not sure why: |
I'm AFK for the rest of this evening, will try again tomorrow |
Thanks! Answered "over there" to keep things together. |
Merge request with F-Droid was accepted, OpenAthena for Android may be showing up there soon. Marking this issue as closed |
@mkrupczak3 Congrats! May I, in this context, repeat my question from above: Do you want me to keep OpenAthena in my repo once it is available at F-Droid.org? Usually I remove apps at that stage (after a decent overlap of 10..14 days). But on request by their authors, especially with reproducible builds being established, keeping it here is an option if needed. |
@IzzySoft I don't see any problem with keeping it available on IzzySoft, and in fact it might improve overall availability of the app I'll put an F-Droid badge with the others when the app appears there |
As I wrote, it's technically possible if requested. Say you want me to keep it, and I mark it accordingly (else it surfaces for removal once it shows up at F-Droid). |
Please keep it |
Sure, I can review that either as a pull request or just the raw text if you prefer. The app hasn't shown up on F-Droid yet, I'm thinking might be because I had a changelog present for version code 27 but not 28. I just pushed out a bug hotfix with version code 29, with any luck it will show up on F-Droid soon |
Maybe I should add German localization soon, as the 9th supported language? @IzzySoft Would you be interested in helping with a QA review once a localization is ready? |
Index publish is just pending. I didn't look which apps where built this cycle, but we should know pretty soon™ – as soon as signing is done and the index published.
Shall I make a PR with what I have at my end, to combine both your comments? Will do. I'll also resolve the "circular reference" then (fulldesc sends readers to your homepage for more information – but all currently found there is basically a link to the Github repo were said details have to be searched for; I've linked that directly. Also, my descriptions are HTML "compressed" to one line; reason for the latter is that otherwise F-Droid replaces each There you go 😃 |
PS: monthly quality checker here just complained:
Fastlane limit there is 500 chars, everything behind that will simply be cut off, even mid-word. When taking a look at that changelog if I could bring it back into limits (I succeeded heavily there) I saw you're using HTML there. Well, that's not permitted – only plain-text. Hence the results look pretty ugly. Please use plain-text (and not more than 500 chars) there with future releases. Thanks! |
full_description.txt
andshort_description.txt
forzh-CN
are empty (0 byte) files which causes some issues: the "empty" description might overwrite the fallbacken-US
one causing the app to show up without any description for that locale – or as in my case, it throws an "failed download: empty file" error on each update.So could you please either fill those files with content, or remove them entirely? Thanks in advance!
Oh, and btw: you might wish to pick a badge to accompany the lonely PlayStore one, linking it to
https://apt.izzysoft.de/packages/com.openathena/
😉The text was updated successfully, but these errors were encountered: