getActivity / AndroidProject-Kotlin

Android 技术中台 Kotlin 版本,但愿人长久,搬砖不再有
Apache License 2.0
1.35k stars 242 forks source link

proguard 配置引入 proguard-base.pro 打出来的包会比 getDefaultProguardFile('proguard-android-optimize.txt') 大 1M+ #44

Closed syxc closed 11 months ago

syxc commented 11 months ago

建议收集

getActivity commented 11 months ago
image image image image image
getActivity commented 11 months ago

小伙子,我验证了一下,并没有出现你说的包体能小 2M 的问题。

syxc commented 11 months ago

感谢阿轮细心的验证,容我我回头再测几次再来反馈,我的环境:

syxc commented 11 months ago

配置 1:

defaultConfig {
  // 模块混淆配置
  consumerProguardFiles 'proguard-base.pro'
}
proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-sdk.pro', 'proguard-app.pro'

WX20231219-222610@2x

配置 2:

defaultConfig {
  // 模块混淆配置
  // consumerProguardFiles 'proguard-base.pro'
}
proguardFiles getDefaultProguardFile('proguard-android-optimize.txt'), 'proguard-sdk.pro', 'proguard-app.pro'

getActivity commented 11 months ago
image
getActivity commented 11 months ago

另外你确定把 proguard-base.pro 注释掉没有什么问题?里面是一些不能被混淆类的规则。

syxc commented 11 months ago

另外你确定把 proguard-base.pro 注释掉没有什么问题?里面是一些不能被混淆类的规则。

我理解你的意思,所以我提到了一个前提,在不影响正常功能的情况,有没有优化的可能,抛砖引玉~

getActivity commented 11 months ago

@syxc 小伙子,我这两天忙于工作,不是特别有空,我星期六日找时间研究一下,但是为了保险起见,建议你不要注释 proguard-base.pro,避免混淆出现 Bug,等我研究出来结果了会在这个 issue 下面同步你。

syxc commented 11 months ago

@syxc 小伙子,我这两天忙于工作,不是特别有空,我星期六日找时间研究一下,但是为了保险起见,建议你不要注释 proguard-base.pro,避免混淆出现 Bug,等我研究出来结果了会在这个 issue 下面同步你。

明白,工作优先。

getActivity commented 11 months ago
image
getActivity commented 11 months ago
image
getActivity commented 11 months ago

小伙子,我查出来问题的根源,是这个混淆规则写得有问题,目前我已经将这条混淆规则去除掉,你可以验证一下,另外建议你再去除掉这个规则之后,进行全方位的自测及测试的验证后再上线。

syxc commented 11 months ago

小伙子,我查出来问题的根源,是这个混淆规则写得有问题,目前我已经将这条混淆规则去除掉,你可以验证一下,另外建议你再去除掉这个规则之后,进行全方位的自测及测试的验证后再上线。

好的,多谢轮子哥🤝