-
Notifications
You must be signed in to change notification settings - Fork 319
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
FDroid release #110
Comments
Yes, but have not time to study how to do that. |
Yeah that's what I'm doing just now, I was just wondering about the future for updating and things. Thanks for the quick response, I'm glad things are going well for you with this :-) |
F-Droid maintainer here. Unfortunately, we've run into a lot of problems, so we currently cannot build this app:
(see here for details) Both violates our inclusion citeria. Is there a chance to solve that, @foobnix? |
Sorry but my app can't work without those jars and google play libraries. |
Thanks for your answer, though it is not what we had hoped for. I'll then have to close the RFP at F-Droid, as we cannot build this app in this situation. By the way, there probably would be a way around those JARs: most of them seem to come from free software projects, so those could possibly be included as git submodules. But no such thing for the Google Play stuff. Just wondering why an eBook reader should depend on such? What is it you need from GMS that is essential? Seeing the |
Yes, my app used a lot third party components, like Recycle View or Card View
Yes, this flow for PRO version without Ads (cut the ads functionality :) You can build an app without google play ads using this dependency project. If you are ok with all of this dependencies, I can help you write the build script that can in one click download and prepare apk for PRO version or you can take it here Ivan |
Thanks, @foobnix! I've asked our "integrators" to check. To me this looks as if it could work – but never having built a single app, I cannot tell for sure. |
PRO version is without play-services, I don't see other violations |
Neither do I (if it is OK for you that we have it in F-Droid). Just thinking about "too complex builds" concerning those libraries, but as a non-dev I hardly can tell. @Rudloff could throw in his opinion, that would be more important than mine here. |
Even if we can build a version without Play services, there is still the problem of the multiple prebuilt JARs.
|
I will prepare build script that also fetch libraries from maven. |
Cool, thanks! Best would AFAIK be Maven Central or JCenter. Note that not all maven repos are "trusted" by F-Droid. There's a list somewhere in the code of fdroidserver… Ah, here it is: allowed_repos = [re.compile(r'^https?://' + re.escape(repo) + r'/*') for repo in [
'repo1.maven.org/maven2', # mavenCentral()
'jcenter.bintray.com', # jcenter()
'jitpack.io',
'www.jitpack.io',
'repo.maven.apache.org/maven2',
'oss.jfrog.org/artifactory/oss-snapshot-local',
'oss.sonatype.org/content/repositories/snapshots',
'oss.sonatype.org/content/repositories/releases',
'oss.sonatype.org/content/groups/public',
'clojars.org/repo', # Clojure free software libs
's3.amazonaws.com/repo.commonsware.com', # CommonsWare
'plugins.gradle.org/m2', # Gradle plugin repo
'maven.google.com', # Google Maven Repo, https://developer.android.com/studio/build/dependencies.html#google-maven
]
] Uh, and forget about the |
@Rudloff @IzzySoft |
I have also added (get jars from maven) it looks like repo.spring.io and github.com is missing in your allow jars list |
It looks like okhttp-digest is also available on jCenter: https://jcenter.bintray.com/com/burgstaller/okhttp-digest/1.18/okhttp-digest-1.18.jar |
@Rudloff I have fix update_jar.sh script |
Another problem is that the dandar3 repositories you are using contain prebuilt JARs and we don't allow that (we have no way to know if the libraries have been modified). Overall, I think it would be easier if you were using a package manager like Maven or Gradle. I also noticed that cloudrail-si-android is not open source, so we can't include it. (But I'm a bit surprised, I thought Maven Central contained open source libraries.) |
Thanks @foobnix! But now we have another problem: betaImplementation 'com.google.android.gms:play-services-ads:16.0.0'
libreraImplementation 'com.google.android.gms:play-services-ads:16.0.0'
pdf_v2Implementation 'com.google.android.gms:play-services-ads:16.0.0'
ebookaImplementation 'com.google.android.gms:play-services-ads:16.0.0'
tts_readerImplementation 'com.google.android.gms:play-services-ads:16.0.0'
pdf_classicImplementation 'com.google.android.gms:play-services-ads:16.0.0' GMS is proprietary, and not permitted in F-Droid. Could you make a build flavor coming without that? Further I'm not sure about Cloudrail. I vaguely remember issues with that; @Rudloff might be able to tell. |
@IzzySoft There is a |
@Rudloff I just wondered as the quoted part seems to be in the "general" dependencies block. I assume the "prefix" on the "implementation" keyword (e.g. "betaImplementation" → "beta") then does address the flavor – so only "fdroidImplementation" applies to the fdroid flavor, correct? In that case: 🙊 – and thanks for the hint! |
Thanks @foobnix! For build relevant things, I can't answer – Pierre must say. |
I tried building 01d24f8 and using Gradle definitely makes it easier for us, thanks! However, I have another problem. We remove binary files before build and
I see the cource for this library is in the repository, but I'm not sure how to build it. |
@Rudloff please find an updated build (included sources of simd) |
I tried 9ec9df1 and now the NDK stuff builds correctly, but I get this error:
|
@foobnix The version issue still stands. Eg. since you don't have static version* we can't use that to pick up new releases, we need to pick them up from elsewhere where the version is more or less one step ahead, and no corresponding tag to build, so the main build server just picks up unbuildable imaginary releases. |
@licaon-kter
|
F-Droid reads 8.3.33 since your The problem is that you modify the version ahead of time. The ideal situation would be that you modify version* when you Tag a release. Adding yet another variable might help, but won't that be annoying to you? Maybe even forgotten from time to time? :) |
/close this issue as the app is actually published |
I will add FDroidCodeNumber and FDroidVersionNumber to the next tag 8.3.36 |
@licaon-kter |
@foobnix Why do you keep updating the versionName/Code for F-Droid if you never Tag actual releases? Ref: https://f-droid.org/wiki/page/com.foobnix.pro.pdf.reader/lastbuild_3780 |
@licaon-kter I think you check tags only for a new release. I update the version for testers but the release is not ready. |
The checkupdate step is just a grep, if it finds something updated it thinks you've just updated and imediately later tagged. If you update now and tag tomorrow, running the check now will add that new version...but of course it can't build. |
If we use Tags we need static versionName/Code. If we use HTTP (as we do for this app) we grep a file for that versionName/Code...and we trust that a proper Tag exists. |
I'll open an issue so maybe we can combine them or smth. Also seen here: MarcusWolschon/osmeditor4android#1163 |
So anyone could explain why fdroid build is months old |
Is none one updating it orrr |
Is it the last update |
So since they can't rebuild it due to errors, that is the only version on fdroid |
Dame |
Any reason why this issue is still open? It seems to be resolved as the F-Droid version is up-to-date. |
@yephny F-Droid version isn't up-to-date now again and maybe therefore there is no point in closing this issue when issue keeps repeating. |
@ikanakova Oh, I thought it was just lagging behind, which isn't uncommon in F-Droid. |
Looks like this issue is completed and should be closed? I cannot thank you enough @foobnix for your efforts to provide a clean version of your reader! Do you mind if I kindly ask you to provide different donation opportunities (LiberaPay/OpenCollective/Github) in this repo's README and also maybe... your website. |
Yup, can close. And reopen in the future as needed :) |
Yep seems like it, but it seems like the f-droid variant has some limitations. Can you include the PRO variant in your repo @IzzySoft? https://github.com/foobnix/LibreraReader/releases/download/8.8.76/Librera.Pro-8.8.76-uni.apk |
@rollingmoai "seems"? Which ones? |
https://f-droid.org/en/packages/com.foobnix.pro.pdf.reader
|
Ah, right, the stuff that's not free software or open-source. |
Nope. For one, see the reasons @licaon-kter just gave (guess the collection of "offenders" would go beyond limits). And for the other, speaking of limits: file size is already twice the per-app limit for my repo. |
Are you planning an fdroid release?
The text was updated successfully, but these errors were encountered: