oxyroid / M3UAndroid

Open Source Player, which made of jetpack compose. Android 8.0 and above supported.
https://t.me/m3u_android
Apache License 2.0
216 stars 24 forks source link

F-Droid not reproducible #95

Closed linsui closed 11 hours ago

linsui commented 3 months ago

Describe the bug See https://gitlab.com/linsui/fdroiddata/-/jobs/6195627756. The library list generated by aboutLibraries in your apk is empty.

oxyroid commented 3 months ago

I think that because I enabled the offline-mode of aboutLibraries. Now I have disable it and the generated list is appeared in my smartphone.

linsui commented 3 months ago

Then I just wait for the next version? Do you know why the list is generated in our build?

linsui commented 3 months ago

Since you enabled splitAbi, should we also do that?

oxyroid commented 3 months ago

I think abi split is necessary because the fully package is a little big

oxyroid commented 3 months ago

Yes, just waiting for next release

linsui commented 3 months ago

We need different version codes for different abis, i.e. 1060 for universal and 1061, 1062, 1063, 1064 for different abis.

oxyroid commented 3 months ago

Well, then let’s not make abi split for f-droid.

linsui commented 3 months ago

:ok_hand:

linsui commented 2 months ago

It fails again. There is an item for com.github.anilbeesetti.nextlib:nextlib-media3ext in our apk but not yours.

oxyroid commented 2 months ago

I have made "codec" flavor, the lite codec one has not nextlib item but the rich codec has. Did you compare the apk with the right flavor? richCodec + stableChannel

linsui commented 2 months ago

There is no other difference so I thought it's the correct flavor.

oxyroid commented 2 months ago

Which apk file in the release page you compared with?

linsui commented 2 months ago

https://github.com/realOxy/M3UAndroid/releases/download/v1.14.0-beta08/1.14.0-beta08_universal.apk

oxyroid commented 2 months ago

Can you provide the compare result?

linsui commented 2 months ago

1.14.0-beta08_universal.apk.html.zip

oxyroid commented 2 months ago

Oh, you mean about-library. I will replace it.

linsui commented 2 months ago

Ah, sorry, I didn't make it clear...

licaon-kter commented 3 days ago
│ │ │   --- 1.14.0-beta13_universal/smali/mb/a.smali
│ │ ├── +++ com.m3u.androidApp_139/smali/mb/a.smali
│ │ │ @@ -24,19 +24,19 @@
│ │ │  .method public constructor <init>(Landroid/app/Application;)V
│ │ │      .locals 2
│ │ │  
│ │ │      invoke-direct {p0}, Ljava/lang/Object;-><init>()V
│ │ │  
│ │ │      iput-object p1, p0, Lmb/a;->a:Landroid/app/Application;
│ │ │  
│ │ │ -    const-string p1, "1.14.0-beta11"
│ │ │ +    const-string p1, "1.14.0-beta13"
│ │ │  
│ │ │      iput-object p1, p0, Lmb/a;->b:Ljava/lang/String;
│ │ │  
│ │ │ -    const/16 p1, 0x89
│ │ │ +    const/16 p1, 0x8b
│ │ │  
│ │ │      iput p1, p0, Lmb/a;->c:I
│ │ │  
│ │ │      const-string p1, "stableChannelRichCodec"
│ │ │  
│ │ │      const-string v0, "snapshotChannel"

for https://github.com/oxyroid/M3UAndroid/releases/tag/v1.14.0-beta13