QiuSimons / openwrt-mos

GNU General Public License v3.0
229 stars 89 forks source link

我打算重新把shell脚本里的那一堆硬编码的玩意解耦。 #141

Closed daiaji closed 6 months ago

daiaji commented 1 year ago

感觉晚弄不如早弄,我有时间就搞一下吧。 这感觉不搞,迟早变屎山。 虽然现在那些为了复用而整出来的函数,和意味不明的函数名。 看起来就蛮吓人了。

daiaji commented 1 year ago

目前想到一个点子,由于几乎所有脚本都调用过那个类库脚本,这脚本里声明全局变量的话,其他脚本应该都能继承这里的变量,硬编码的DNS和下载反代URL之类,应该就也能变量化了。

在开头就声明变量也还蛮常见的吧。 万一要改应该也方便点。

QiuSimons commented 1 year ago

可以的,大佬加油!

daiaji commented 1 year ago

https://github.com/QiuSimons/openwrt-mos/pull/142 OK了。 先审查一下代码吧。

QiuSimons commented 1 year ago

大佬辛苦了,各项功能您run test过了么? pr咱先合并了,但最近没啥时间仔细搞,如果大佬后面发现有啥bug,还请继续pr,万分感谢

daiaji commented 1 year ago

set.sh那一堆命令能用&丢到后台并行化吗?

QiuSimons commented 1 year ago

set.sh那一堆命令能用&丢到后台并行化吗?

看了下,理论上应该可以的

daiaji commented 1 year ago

要不你动手? 我在捣鼓PE。

QiuSimons commented 1 year ago

要不你动手? 我在捣鼓PE。

先缓缓,刚升级到v5的核心,先测试一段时间,稳定了再搞景上添花的事情

daiaji commented 1 year ago

快进到摸了(

daiaji commented 6 months ago

历时一年多,终于搞完了。