helge17 / tuxguitar

Improve TuxGuitar and provide builds
Other
469 stars 41 forks source link

Add fork to F-Droid #138

Open licaon-kter opened 1 year ago

licaon-kter commented 1 year ago

ref: https://gitlab.com/fdroid/fdroiddata/-/issues/3101

Code issues: version seems to be stuck and oddly defined in two places: https://github.com/helge17/tuxguitar/blob/master/TuxGuitar-android/AndroidManifest.xml#L5-L6 and https://github.com/helge17/tuxguitar/blob/master/TuxGuitar-android/build.gradle#L15-L16 ( I recommend the latter)

Inclusion policy: https://f-droid.org/en/docs/Inclusion_Policy/ forks need a different appID than original app, and while this could be considered as continued work.. I didn't see any agreement with original dev :(

guiv42 commented 1 year ago

Thanks for your feedback

I didn't see any agreement with original dev :(

For sure we would have liked this agreement also. We tried to contact original dev and proposed help, unfortunately the contact is completely lost :( Last published version of app at SF is about 1.5 year old, and last post I found from original dev was in July'22. Our request was sent 8 months ago now: unfortunately in this case I think forking was the only option to go forward.

Code issues: version seems to be stuck and oddly defined in two places

I'm going to look at this version definition issue.

Just 2 questions:

  1. I understand request from inclusion policy to change app ID, but is it required to change app name also, as we clearly target "continued work"?
  2. Could you please help us identify the "anti-features" mentioned in current version of TuxGuitar in F-Droid, so we can try to fix that?
licaon-kter commented 1 year ago

I understand request from inclusion policy to change app ID, but is it required to change app name also, as we clearly target "continued work"?

appid, app name, icon, I mean TuxGuitar Reborn might be ok :)

Could you please help us identify the "anti-features" mentioned in current version of TuxGuitar in F-Droid, so we can try to fix that?

Think it's more about the cleaning: https://gitlab.com/fdroid/fdroiddata/-/issues/2481

guiv42 commented 1 year ago

OK. Not fully understood the point about "cleaning", but anyway it will only make sense if we can manage the first point.

ClockGen commented 9 months ago

@guiv42 Hello, thanks for continuing to work on TuxGuitar. There's a few things that needs to be done to be "properly" included in F-Droid (old version in F-Droid was basically a one-time build):

guiv42 commented 9 months ago

Thanks @ClockGen for the hints. Let me try to sum up

  1. version remove version from AndroidManifest.xml, only keep version in build.gradle I would keep version defined as "SNAPSHOT" in the source code, and replace it with sed command during build/delivery. This is the way we do it currently, it is very similar to what f-droid build does

  2. gdrive plugin I would vote for its deletion. As of today Android version only represents 2% of downloads for 1.6.0. F-Droid should help reaching more users, so I think removing the "anti-feature" warning is worth this deletion.

  3. fastlane OK, that should be quite easy

  4. rename I would suggest simply "TuxGuitar+", as some apps forks have already done. The most complex task here seems to be the icon redesign

Yes it needs to be done, it's just a matter of priorities. Since this fork started here, we essentially focused on the desktop app, and there's a lot to do for the Android version. Any help welcome, don't hesitate to send PR!

ClockGen commented 9 months ago

I don't know about icon, there are some forks in F-Droid that use the same icon as their original apps (e.g. orgzly revived, syncthing fork) but better ask @licaon-kter about this.

licaon-kter commented 9 months ago

well... do add something to it, a + or a star or whatever :)