Repository navigation
|
Greetings. Pixel 9a, GrapheneOS, Obtainium 1.6.11 I've added Ghost Commander via this Ghost Commander Link supplied by ImranR98 found Here However it states the following:
What settings should I be using to resolve this? |
Replies: 6 comments 7 replies
|
This is a good enough place to ask for help, so no faults regarding GitHub etiquette - as far as I know. I don't know enough otherwise to help without speculating. Maybe the version extraction regex needs to be tinkered with?? Good luck. |
|
I've tried adding it too. I even opened a request, but so far without success. Of course, SourceForge and the project's structure can sometimes cause problems... Let's give it a try together. |
|
I'm not sure if the source needs fixed or what the underlying issue is but ImranR98's import link doesn't work as stated. It's not configured for the %20 before the version number. master-bob is correct, the version trim needs an edit to make this work. Change it to this... Trim version string with ReGex: Edit: |
|
Your solution is almost perfect (or maybe even perfect). Right now, all you need to do is include your code for truncating the version string and specify the $1 match group. The only thing is, I took a look at how your code handles the list of APKs for the release and beta branches, and it seems to me there might be some false positives: |
|
The AI suggested the following options to me, noting that the second one was more accurate:
and
|


I didn't look into this with my initial post above, all I was attempting to do was to revise the version trim to make it work. The $1 match group was already there within ImranR98's import link.
You're also 100% correct. The
.+is greedy meaning it'll pickup any apk that hasCommanderending inapk. This could definitely produce false positives as your screenshots show. So I'll provide a few different revisions to avoid this conflict.Commander.20(\d.*)\.apkCommander.\d{2}(\d.*)\.apkThe
+would still work but the*would be better.