simpletask-android icon indicating copy to clipboard operation
simpletask-android copied to clipboard

Cloudless vs Nextcloud on F-Droid

Open smichel17 opened this issue 7 years ago • 30 comments

I previously had ST Cloudless installed through F-Droid. In F-Droid, it was called simply "Simpletask". The launcher was called "Simpletask Cloudless".

As of the most recent version or two, it now appears as "Simpletask Nextcloud" in F-Droid and "ST Cloudless" in the launcher. The actual installed app remains cloudless, though I have not tried doing a fresh install.

smichel17 avatar Oct 21 '17 16:10 smichel17

I think it's related to f-droid now pulling the apop metadata from this repo's directory structure according to this layout: https://github.com/Triple-T/gradle-play-publisher#play-store-metadata

apparently it falls over the multiple flavours.

Edit: the thing is according to this documentation the values specified in the fdroid metadata should take precedence: https://bubu1.eu/fdroid-website/en/docs/All_About_Descriptions_Graphics_and_Screenshots/#in-the-apps-source-repository

But obviously they don't.

Reported both issues: here and here.

Bubu avatar Oct 23 '17 21:10 Bubu

Is there any reason why the Nextcloud version is not build for F-Droid? I would love to use it.

schiessle avatar Jun 26 '18 20:06 schiessle

@schiessle not any particular reason except that it requires work. Considering you can relatively easy build yourself this is not a priority for me now.

mpcjanssen avatar Jun 28 '18 19:06 mpcjanssen

This is really confusing... I installed "Simpletask Nextcloud" via F-Droid. I thought this had to work with Nextcloud (since it's in the name). It took me an hour before I landed here and realized that I had installed the wrong app.

feluxe avatar Feb 07 '19 00:02 feluxe

@feluxe I agree it's confusing. Unfortunately there is not much I can do about it.

mpcjanssen avatar Feb 07 '19 13:02 mpcjanssen

Seems fixed in current F-Droid version

mpcjanssen avatar Apr 12 '19 17:04 mpcjanssen

@mpcjanssen to clarify: the version which is now in F-Droid does not support Nextcloud but is the standalone version, right?

kirschner avatar Jun 12 '19 13:06 kirschner

Correct

mpcjanssen avatar Jun 12 '19 14:06 mpcjanssen

Thanks for clarifications @mpcjanssen I would still be interested in a version with nextcloud support in f-droid, but I can understand that it is not a priority for you. Thanks for your work!

kirschner avatar Jun 12 '19 17:06 kirschner

Feel free to add it to F-Droid yourself. The only reason for me not to added is lack of time. If someone will make the required fdroiddata PR I am all for it.

mpcjanssen avatar Jun 14 '19 11:06 mpcjanssen

If someone will make the required fdroiddata PR I am all for it.

hi @mpcjanssen, I know this is a very old issue but I still am interested in getting the Nextcloud variant in F-Droid. I may be able to help with the _fdroiddata PR, but would require a lot of your help. I have a couple of queries first:

  • It is unclear to me how are you building the Nextcloud app from the same source and what is added in that build.
  • What about the package name? Do you want to keep it as same as the Play Store one? ie: nl.mpcjanssen.simpletask.nextcloud

fazlerabbi37 avatar Apr 28 '20 08:04 fazlerabbi37

@fazlerabbi37 there is already a PR for the webdav version which can also be used for nextcloud see https://gitlab.com/fdroid/fdroiddata/-/merge_requests/6534

mpcjanssen avatar Apr 28 '20 12:04 mpcjanssen

@mpcjanssen I just copied and pasted your MR, changed the app name to Simpletask Nextcloud and created a new MR: https://gitlab.com/fdroid/fdroiddata/-/merge_requests/6680. Please give it a look and let me know if any improvement is needed.

fazlerabbi37 avatar Apr 28 '20 13:04 fazlerabbi37

Looks like it should work. Note that there are instructions in the fdroiddata repo for verifying that it actually will work (by downloading the fdroid server and trying to run it locally; it's not that hard).

smichel17 avatar Apr 28 '20 15:04 smichel17

@mpcjanssen I just hit a bit of problem regarding the RFP for Nextcloud flavor. Simpletask usages Triple-T structure to provide the description and summery but for the Cloudless app in F-Droid almost all metadata are acompanied in the fdroiddata repo.

My question what should I do for the Nextcloud flavor? I can create a PR to update the Triple-T structure with the data from fdroiddata repo and remove the data from there for Cloudless flavor or just copy and change the fdroiddata repo data for the Nextcloud flavor.

The F-Droid maintainers are pushing (gently) towards adopting either Fastlane or Triple-T file structure as it is less hassle for them and much more easy to maintain for the app developer. IMO, we should go with the first option as you already have Triple-T. Please keep in mind that if we do so, you need to tag and make a new release for it to be picked up by F-Droid.

fazlerabbi37 avatar Jun 01 '20 03:06 fazlerabbi37

If you could create PR for Triple-T that would be great.

mpcjanssen avatar Jun 01 '20 05:06 mpcjanssen

[Meta] it's annoying to me that neither fastlane nor triple-T get everything right.

Triple-T:

  • Metadata inside app folders makes sense
  • Gradle plugin instead of standalone tool is great
  • Metadata structure leaves much to be desired.

Fastlane:

  • Metadata structure works well with multiple locales
  • Changelog structure makes way more sense

Fastlane structure in Triple-T location as a gradle plugin would be amazing.

smichel17 avatar Jun 01 '20 11:06 smichel17

Follow-up: https://gitlab.com/fdroid/fdroidserver/-/issues/829

smichel17 avatar Sep 11 '20 11:09 smichel17

@mpcjanssen I just hard back from F-Droid about this issue. As per this comment, the way fastlane is structured now will not work with F-Droid. In order to get it working, it needs to be flavor-specific. Do you want me to restructure the fastlane directory and send a new PR?

fazlerabbi37 avatar Mar 10 '21 14:03 fazlerabbi37

To clarify, let me quote the referenced comment:

the language/locale directories must reside directly inside fastlane/metadata/android/, no additional level in between.

If you want flavor-specific fastlane, that needs to be placed differently – see the "additional notes" section at the end of my Fastlane Cheat-Sheet: that would then be src/<buildFlavor>/fastlane/metadata/android/ – in your case most likely app/src/(cloudless|dropbox|nextcloud)/fastlane/metadata/android/.

But fastlane/metadata/android/<somethingHere>/en-US (as you currently have it) won't be found – you'd need to drop the /<somethingHere> to make it work.

IzzySoft avatar Mar 10 '21 15:03 IzzySoft

@fazlerabbi37 if you could restructure it, that would be great.

mpcjanssen avatar Mar 11 '21 08:03 mpcjanssen

I will send a PR ASAP.

fazlerabbi37 avatar Mar 11 '21 12:03 fazlerabbi37

Aaah, I also need to get around to the f-droid metadata improvement, it's been on my list for foreeever :S

smichel17 avatar Mar 11 '21 16:03 smichel17

I just created the PR to move Fastlane file to flavor-specific locations #1115.


Now I have a question for @mpcjanssen and @IzzySoft, what happens to the fastlane directory and the files inside them? I am assuming they will stay there, but can we remove the metadata directory inside fastlane directory? I have kept them for now but can update the PR after removing them, if it is ok to do so.

fazlerabbi37 avatar Mar 15 '21 16:03 fazlerabbi37

Just be clear of what you're asking: No need to keep the stuff inside /fastlane (i.e. the previous/old location) for F-Droid, we don't need duplicates; the one in the new location of course needs the metadata (i.e. app/src/*/fastlane/metadata must stay :wink:). When the PR is merged, we should send the bot again on F-Droid's end to see it finally finds it (which might need a tag "on top").

IzzySoft avatar Mar 16 '21 02:03 IzzySoft

@fazlerabbi37 PR merged, thanks.

mpcjanssen avatar Mar 16 '21 11:03 mpcjanssen

update on F-Droid side: as per this comment they need a new release. @mpcjanssen can you please take a look?

fazlerabbi37 avatar Apr 08 '21 10:04 fazlerabbi37

@fazlerabbi37 I have tagged 10.9.1.

mpcjanssen avatar Apr 08 '21 10:04 mpcjanssen

update on F-Droid side: RFP PR is merged! :tada: Hopefully will be build within next week. Thank you, @mpcjanssen, @IzzySoft and everyone!

PS: @mpcjanssen may need additional help from you to do some fix and cleanup after the app is published.

fazlerabbi37 avatar Apr 09 '21 09:04 fazlerabbi37

@fazlerabbi37 thanks a lot. Please let me know if you need something else (log a new issue)

mpcjanssen avatar Apr 09 '21 11:04 mpcjanssen