-
Notifications
You must be signed in to change notification settings - Fork 5.9k
建议将脚本定制部分提取出来 #21
Comments
根目录配置文件中只添加了定制的内容,仅仅只是把.yml中定制部分提取出来了而已,并不会造成任何后期依赖混乱 根目录配置文件不需要命名为.config,任何名字都可以,在.yml中重命名为.config即可,并不会随着克隆到本地而取代默认配置. |
你不觉得这种做法是南辕北辙绕了一圈又回来了吗?
为什么不直接多部署几个flow?
耦合度太高. 高阶用户会直接修改源码.
同第一条
然后你的库里就会多出一堆没用的文件 |
直接修改源码的话,如果lede的源码有更新,自己修改过的文件就没办法pull过来了,会造成很多不可控的问题. |
正如我做的:依官版,做了些hot patch源码和feeds的工作,目前正尝试cache 到底是需要cache哪些目录呢? @KFERMercer 你这V2ray编译要多久?https://github.com/KFERMercer/openwrt-v2ray
需要指定golang版本 - name: Set up Go 1.13
uses: actions/setup-go@v1
with:
go-version: 1.13 |
建议把脚本定制部分的内容提出来放到根目录某个文件中,好处:
1.脚本文件更简洁清晰, 不再需要cat >> .config <<EOF之类的语法,不再需要考虑格式,对齐等,以后同步更新本项目.yml文件也不用考虑定制部分,根目录的配置文件直接按照.config的语法即可,只需要添加需要定制的部分,无需拷贝整个.config文件
2.更方便的编译和管理多个设备的不同固件,只需要在根目录添加几个配置文件即可.
另外建议引入diy.sh,为高阶用户提供更多玩法, 比如https://github.com/garypang13/lede/raw/master/diy.sh
The text was updated successfully, but these errors were encountered: