sbwml / luci-app-alist

LuCI support for Alist
https://github.com/alist-org/alist
1.28k stars 196 forks source link

feat: using upx to optimizing binary size #97

Closed arrio464 closed 6 months ago

sbwml commented 6 months ago

开始的时候加了upx 可选项,但是带来的后果是一些网友吐槽进程启动慢、以及 openwrt 源码需要额外拉取 upx 包,否则构建依赖不足构建失败。https://github.com/sbwml/luci-app-alist/commit/1c3bd91a183ef03438ac175b4dfc1cd149ffb94f

openwrt 自 18.06 开始就已经移除了 upx https://github.com/openwrt/openwrt/commit/0685f2a66cbb4d39417f168cfb37ae6fa7dbf6cb

不动源码的前提的话,估计也就只有lede 是带 upx 了吧

arrio464 commented 6 months ago

开始的时候加了upx 可选项,但是带来的后果是一些网友吐槽进程启动慢、以及 openwrt 源码需要额外拉取 upx 包,否则构建依赖不足构建失败。1c3bd91

openwrt 自 18.06 开始就已经移除了 upx openwrt/openwrt@0685f2a

不动源码的前提的话,估计也就只有lede 是带 upx 了吧

我觉得应该给用户一个选择的权力,默认不启用upx就行 至于额外拉取 upx 包,或许也并非不可接受(? 反正clean build动辄几个小时,也不差几分钟来编译upx