Acode icon indicating copy to clipboard operation
Acode copied to clipboard

F-Droid builds are out of date

Open balporig opened this issue 3 years ago • 17 comments

The current F-Droid version getting quite old 1.4.164 (164) - Added on 2021-10-12

balporig avatar Jun 04 '22 16:06 balporig

Very out-of-date. @deadlyjack, are you still planning to support F-droid in the future? Planning to move away from open source?

jstetten avatar Jun 08 '22 07:06 jstetten

I don't know how to update there. I have no idea. If anyone can help.

deadlyjack avatar Jun 08 '22 07:06 deadlyjack

Since ACode is the premiere (IMO) open source coding app and central to the flow of a great deal of open-source-minded developers, I'd like to ask for @IzzySoft's guidance, when he has time — as he's the font of all Android open source repo knowledge.

jstetten avatar Jun 08 '22 19:06 jstetten

I can take a look (which would have been easier if I had known the package name :wink:). Seems like F-Droid is no longer able to build the app:

  • disable: webpack fails
  • disable: Task 'assembleRelease' not found in project ':app'.

1.4.164 was the last version with a successful build. 1.5.4 failed with the above errors, 1.5.5 was not even tried as at the tag it still identifies as 1.5.4 (i.e. version was only adjusted AFTER tagging, should have been BEFORE; so lifting the tag to the corresponding commit would trigger another build).

he's the font of all Android open source repo knowledge.

:flushed: Oops…

IzzySoft avatar Jun 08 '22 21:06 IzzySoft

Do you know how to fix YAML prebuild section? There are some corrections.

prebuild:
      - mkdir -p platforms
      - npm install
      - node ./utils/config.js p paid
      - ./node_modules/.bin/webpack --progress --mode production
      - cordova platform add android # change to android@9
      - node ./utils/loadStyles.js
      - sed -i -e '/signingConfig signingConfigs.release/d' -e '/gms/d' -e '/core-google-shortcuts/d'
        platforms/android/app/build.gradle

deadlyjack avatar Jun 12 '22 07:06 deadlyjack

No, sorry – I'm not a developer, and completely unfamiliar with node stuff (so for me it would rather be copy-pasting blindly). I couldn't even test that or, if it throws errors, guess what they mean and how to fix. Maybe you could open a merge request with that at fdroiddata? You would most likely know how to interpret output. Should you need help there then, a team member more familiar with node would certainly be able to help.

IzzySoft avatar Jun 12 '22 09:06 IzzySoft

...Did http://github.com/deadlyjack/Acode/pull/430 remediate this?

RokeJulianLockhart avatar Jul 29 '22 13:07 RokeJulianLockhart

No idea. The last tag I see here is 1.6.0, which was before that PR was merged; the corresponding build is disabled at F-Droid's end:

  - versionName: 1.6.0
    versionCode: 186
    disable: webpack fails
    commit: 0b5dd6d3b723ac4f1d2a8f591af3541d603106b1

We will learn more when the next tag was picked up. And to repeat myself: I cannot test this, I'm no developer :wink: But maybe I find someone to check, no promises though.

IzzySoft avatar Jul 29 '22 15:07 IzzySoft

download and view the latest build log for version 1.6.0: https://f-droid.org/repo/com.foxdebug.acode_186.log.gz

look at the lines where there are the words ERROR. there seems to be a problem with the file: /src/components/inputHints.js sometimes he does not see the extension, sometimes the letter case is different.

OmlineEditor avatar Jul 31 '22 23:07 OmlineEditor

Thank you so much for the report. I have fixed this. It was supposed to be '../inputhints' instead of '../inputHints'. It might work in the next update.

deadlyjack avatar Jul 31 '22 23:07 deadlyjack

Building 05acef9d1c9d47fae25055948a2e133074ddae27 fails with: com.foxdebug.acode_196.log.gz

licaon-kter avatar Aug 01 '22 09:08 licaon-kter

Building 05acef9 fails with: com.foxdebug.acode_196.log.gz

now there is only one error and the error is strange

2022-08-01 09:43:51,786 ERROR: Could not build app com.foxdebug.acode: Error running prebuild command for com.foxdebug.acode:1.6.0

perhaps you need to change the version from 1.6.0. to 1.6.1 and upload new corrected sources code?

OmlineEditor avatar Aug 01 '22 17:08 OmlineEditor

There's no 1.6.1 in your code: https://github.com/deadlyjack/Acode/blob/main/platforms/android/app/src/main/AndroidManifest.xml#L2

Where are the "corrected sources code" if not here in this repo? :)

now there is only one error and the error is strange

That's not the error, that's the consequence.

licaon-kter avatar Aug 01 '22 17:08 licaon-kter

Where are the "corrected sources code" if not here in this repo? :)

it seems to me that f-droid does not download the repository, it downloads the archive from the list of releases. are you sure that the entire repository is being downloaded?


I compared the contents of the files from this archive and from the repository entirely, and they are different. https://github.com/deadlyjack/Acode/archive/refs/tags/v1.6.0.zip I think it's easier to release a new version 1.6.1 for testing.

there should be a new archive that will take f-droid: https://github.com/deadlyjack/Acode/tags

OmlineEditor avatar Aug 01 '22 21:08 OmlineEditor

F-Droid clones the repo at the right commit one wants. Eg. 05acef9d1c9d47fae25055948a2e133074ddae27

Not sure I follow, what is missing exactly? Link to the missing files?

licaon-kter avatar Aug 02 '22 04:08 licaon-kter

Not sure I follow, what is missing exactly? Link to the missing files?

all files are there but differ in content in the repository and in the archive here https://github.com/deadlyjack/Acode/archive/refs/tags/v1.6.0.zip

I thought this particular archive was used to build

OmlineEditor avatar Aug 02 '22 14:08 OmlineEditor

I've heard that github archives sometimes (not sure why) have different contents.

But no, we always use git to clone the whole repo, then checkout the needed commit.

licaon-kter avatar Aug 02 '22 14:08 licaon-kter

Since ACode is the premiere (IMO) open source coding app and central to the flow of a great deal of open-source-minded developers, I'd like to ask for @IzzySoft's guidance, when he has time — as he's the font of all Android open source repo knowledge.

it's not that big of a deal, you're not going to code anything serious on your phone lol

griffi-gh avatar Aug 13 '22 12:08 griffi-gh

https://gitlab.com/fdroid/fdroiddata/-/merge_requests/11628 Finally it builds and works. I still need to patch the code but thanks to https://github.com/deadlyjack/Acode/pull/430 some patches are not needed.

linsui avatar Aug 29 '22 03:08 linsui

Thanks for the help, I appreciate it.

deadlyjack avatar Sep 18 '22 00:09 deadlyjack

Close this then

licaon-kter avatar Sep 18 '22 06:09 licaon-kter

Seems like it's broken again- last version 1.6.0

I thought development had ceased until just now.

AeliusSaionji avatar Mar 18 '23 22:03 AeliusSaionji

@AeliusSaionji there's no release nor any tags, maybe they existed in the past so F-Droid could magically update... but they ain't now :(

licaon-kter avatar Mar 18 '23 22:03 licaon-kter

Updated. https://gitlab.com/fdroid/fdroiddata/-/merge_requests/12763

linsui avatar Mar 19 '23 06:03 linsui

Can we have the minor updates as well? Pretty please 🥺

Beryesa avatar Jun 25 '23 22:06 Beryesa

Updated. https://gitlab.com/fdroid/fdroiddata/-/merge_requests/13286 I have to take some change from https://github.com/deadlyjack/Acode/pull/659.

linsui avatar Jun 26 '23 11:06 linsui