OpenWrtManager icon indicating copy to clipboard operation
OpenWrtManager copied to clipboard

Inclusion in F-Droid

Open buongiorgio opened this issue 2 years ago • 23 comments

Hello, you already said you're not going to add the app in the F-Droid repository https://github.com/hagaygo/OpenWRTManager/issues/12 F-Droid policy permits also to someone who isn't the author to propose the inclusion of an app, if the original author does not oppose. So this issue is just to ask you if you are OK with me proposing the inclusion of your app in F-Droid.

buongiorgio avatar Jun 11 '23 10:06 buongiorgio

No objection here.

hagaygo avatar Jun 11 '23 11:06 hagaygo

@IzzySoft is any progress on this? That would be really great

yurtpage avatar Apr 28 '24 20:04 yurtpage

I cannot tell, I'm currently not active at F-Droid.

IzzySoft avatar Apr 28 '24 20:04 IzzySoft

@yurtpage but you can add my repo to your F-Droid Client, where OpenWrtManager is listed.

@hagaygo if you wish, you can pick a badge to link there from your Readme, so that Google one doesn't look so loneley :wink:

IzzySoft avatar Apr 28 '24 20:04 IzzySoft

@IzzySoft that's sad to hear. Maybe still you may help to create the app descriptor for inclusion? That would help to get the app to the store and attract more users and contributors.

yurtpage avatar Apr 29 '24 20:04 yurtpage

I can send a PR with the fastlane structures if @hagaygo wants it, they are set up at my end anyway.

IzzySoft avatar Apr 29 '24 22:04 IzzySoft

As far i can understand inclusion in F-Droid is awaiting donating ?

https://gitlab.com/fdroid/rfp/-/issues/2442

hagaygo avatar Apr 30 '24 05:04 hagaygo

@IzzySoft you already sent it, ty. We need the yaml file https://gitlab.com/fdroid/fdroiddata/-/blob/master/templates/build-flutter.yml

@hagaygo that's optionally, more like to filter out garbage

yurtpage avatar Apr 30 '24 06:04 yurtpage

inclusion in F-Droid is awaiting donating ?

No, that is not mandatory, it's rather a hint to the possibility.

We need the yaml file

Sorry, can't help you with that. I just run a binary repo, so I don't have all the build stuff in. The parts from my YAML that would be useful there are only these:

Categories:
  - Connectivity
License: GPL-3.0-only
AuthorName: Hagay Goshen
AuthorEmail: [email protected]
SourceCode: https://github.com/hagaygo/OpenWrtManager
IssueTracker: https://github.com/hagaygo/OpenWrtManager/issues
Changelog: https://github.com/hagaygo/OpenWrtManager/releases

AutoName: OpenWRT Manager

// here go the build details

AutoUpdateMode: Version
UpdateCheckMode: Tags

IzzySoft avatar Apr 30 '24 07:04 IzzySoft

@buongiorgio @hagaygo please answer https://gitlab.com/fdroid/rfp/-/issues/2442#note_1886854654

yurtpage avatar Apr 30 '24 08:04 yurtpage

Normally i use the latest stable one.

3.19 currently.

hagaygo avatar Apr 30 '24 08:04 hagaygo

@hagaygo Please join the discussion https://gitlab.com/fdroid/rfp/-/issues/2442#note_1886893631

yurtpage avatar Apr 30 '24 08:04 yurtpage

great, but with 3.7.12 or 3.19.6, it fails to build: https://gitlab.com/fdroid/rfp/-/issues/2442#note_1886893631

licaon-kter avatar Apr 30 '24 08:04 licaon-kter

great, but with 3.7.12 or 3.19.6, it fails to build: https://gitlab.com/fdroid/rfp/-/issues/2442#note_1886893631

The log start with missing files at this point :

"Install Android SDK Platform 34 (revision: 3)" complete.
"Install Android SDK Platform 34 (revision: 3)" finished.
lib/Page/mainPage.dart:9:8: Error: Error when reading 'lib/OpenWrt/Model/AuthenticateReply.dart': No such file or directory
import 'package:openwrt_manager/OpenWrt/Model/AuthenticateReply.dart';
       ^
lib/Page/mainPage.dart:10:8: Error: Error when reading 'lib/OpenWrt/Model/CommandReplyBase.dart': No such file or directory
import 'package:openwrt_manager/OpenWrt/Model/CommandReplyBase.dart';
       ^

It seems your build does not find the app files from some reason.

hagaygo avatar Apr 30 '24 09:04 hagaygo

@hagaygo

It seems your build does not find the app files from some reason.

yes, Linux is case sensitive, lib/OpenWrt vs https://github.com/hagaygo/OpenWrtManager/blob/github/lib/OpenWRT :smile:

licaon-kter avatar Apr 30 '24 11:04 licaon-kter

Might be the problem, i currently don't have access to a working flutter sdk environment for testing it on Linux and fixing the case sensitivity.

The folder you mentioned might be the only problem , i guess its a by product of https://github.com/hagaygo/OpenWrtManager/commit/f59b8f82c591508b419953c57ea842edc3727c0d

hagaygo avatar May 01 '24 11:05 hagaygo

maybe you can make a case sensitive search in logs

yurtpage avatar May 01 '24 13:05 yurtpage

maybe you can make a case sensitive search in logs

You are welcome to fork and try until it works on the fork while testing it with @licaon-kter and then create a pull request.

Currently i can test only on windows 🤷‍♂️

hagaygo avatar May 01 '24 13:05 hagaygo

@licaon-kter

Hi , managed to setup a working linux development using docker

sudo docker run --name flutter --rm -v <hostAppDirectory>:/home/mobiledevops/app -it mobiledevops/flutter-sdk-image:3.19.4

Got the same errors you mentioned and fixed the filename case on https://github.com/hagaygo/OpenWrtManager/commit/d585f99f90ca9b33298d1ae6c0043dde04804234

Please try again.

Thanks.

hagaygo avatar Jun 15 '24 08:06 hagaygo

@hagaygo builds fine

reproducible builds ( https://f-droid.org/docs/Inclusion_How-To/#reproducible-builds ) are not possible since you build on windows and Flutter embeds its code paths ( https://f-droid.org/docs/Reproducible_Builds/#embedded-build-paths )

on macOS/Linux we can use the same path, but yours is C:/Users/hagay/source/repos/...

for autopupdates we need a place to find the exact flutter version, eg. have Flutter as a git submodule that you use during builds too so it's always up to date, or at least have it written somewhere so we can grep it

licaon-kter avatar Jun 18 '24 11:06 licaon-kter

for autopupdates we need a place to find the exact flutter version, eg. have Flutter as a git submodule that you use during builds too so it's always up to date, or at least have it written somewhere so we can grep it

Hi

Placing the whole flutter SDK (even as submodule) in the repo seems wrong.

Adding a file called flutter.version in the repo root, which contains the flutter version is used ( 3.19.6 for example) would be enough ?

hagaygo avatar Jun 19 '24 06:06 hagaygo

Yes, just make sure to keep it up to date

licaon-kter avatar Jun 19 '24 06:06 licaon-kter

@licaon-kter just added the file , please let me know if you need anything else.

hagaygo avatar Jun 21 '24 09:06 hagaygo

done: https://gitlab.com/fdroid/fdroiddata/-/commit/630344504218db4de351998385592e537eb661a0 :tada:

licaon-kter avatar Jul 23 '24 16:07 licaon-kter

Thank you!

23 июля 2024 г. 19:00:28 GMT+03:00, Licaon_Kter @.***> пишет:

done: https://gitlab.com/fdroid/fdroiddata/-/commit/630344504218db4de351998385592e537eb661a0 :tada:

-- Reply to this email directly or view it on GitHub: https://github.com/hagaygo/OpenWrtManager/issues/37#issuecomment-2245638402 You are receiving this because you were mentioned.

Message ID: @.***> Page

yurtpage avatar Jul 24 '24 03:07 yurtpage

Thank you @licaon-kter @hagaygo The issue can be closed

yurtpage avatar Jul 30 '24 04:07 yurtpage

@hagaygo as you might have missed it above: will you pick a badge to link to OpenWrtManager at IzzyOnDroid from your Readme? Looks like that badge is still missing there :wink: Thanks!

IzzySoft avatar Sep 08 '24 09:09 IzzySoft

@IzzySoft Open a PR with what suppose to be missing and I'll check it out later on.

hagaygo avatar Sep 08 '24 12:09 hagaygo

There you go :smiley:

IzzySoft avatar Sep 08 '24 13:09 IzzySoft

Interesting placement...

licaon-kter avatar Sep 08 '24 13:09 licaon-kter