Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

致所有使用 KFERMercer/OpenWrt-CI 的人: #3205

Closed
KFERMercer opened this issue Feb 19, 2020 · 12 comments
Closed

致所有使用 KFERMercer/OpenWrt-CI 的人: #3205

KFERMercer opened this issue Feb 19, 2020 · 12 comments

Comments

@KFERMercer
Copy link
Contributor

KFERMercer commented Feb 19, 2020

KFERMercer/OpenWrt-CI 是给正在维护自己的openwrt的库并且知道自己在做什么的人用来测试的测试环境, 而不是让只想得到固件文件的普通用户用来编译的工具.

建议所有自认为是后者的人停止使用 KFERMercer/OpenWrt-CI, 转而去使用 P3TERX/Actions-OpenWrt.

@P3TERX 创建的 workflow 不需要 fork 整个 OpenWrt 存储库, 只需要上传.config文件即可完成编译, 而且支持使用 SSH 连接 GitHub Actions 服务器, 更加直观的控制整个编译过程.

同时请所有尚不明白 Pull request 为何物的人, 在学习过网上随处可见的Git教程后, 停止向 coolsnowwolf/lede 以及 openwrt/openwrt 推送任何垃圾合并请求.

@lqhorochi
Copy link

不用的路过
用自己电脑编译.

@adaaaaaa
Copy link

我两者都有怎么办??难道有人任意一次编译了之后都不下载固件的吗?🤪🤪🤪

@9l
Copy link
Contributor

9l commented Feb 20, 2020

我两者都有怎么办??难道有人任意一次编译了之后都不下载固件的吗?🤪🤪🤪

不是不让下载固件,是不要瞎鸡脖的pr辣鸡配置文件给别人,你是拉新代码,不是推送辣鸡代码

@segadrive
Copy link

我pork了不推送只修改不影响把?小白真不懂

@z826540272
Copy link
Contributor

我早就用P3TERX/Actions-OpenWrt了,我觉得你的脚本很难用

@z826540272
Copy link
Contributor

垃圾合并请求主要是因为github的辣鸡设计.

@z826540272
Copy link
Contributor

我两者都有怎么办??难道有人任意一次编译了之后都不下载固件的吗?🤪🤪🤪

不是不让下载固件,是不要瞎鸡脖的pr辣鸡配置文件给别人,你是拉新代码,不是推送辣鸡代码

github的界面设计拉去代码比推送代码复杂

@kiddin9
Copy link

kiddin9 commented Feb 20, 2020

我早就用P3TERX/Actions-OpenWrt了,我觉得你的脚本很难用

同感

diy.sh default-settings patch files大法以及文件覆盖等都可以用来调整维护 适合高阶玩家,哪需要拉取整个项目,在OpenWrt-CI中反而把diy.sh files ssh这些高阶玩法都去掉了.如果不是大规模修改那用p3terx的就完全够用了

把.config写在.yml中更是画蛇添足
.yml空格换行回车格式要求严格 .config放到.yml中还得按照cat >> .config <<EOF之类的语法
增加了操作难度和出错可能性,反而不方便维护了,误导小白呀

@coolsnowwolf coolsnowwolf pinned this issue Feb 20, 2020
@Zxilly
Copy link

Zxilly commented Feb 21, 2020

github这个fork之后,如果在网页端操作合并,很容易把merge发成PR。。。

@ch1993128
Copy link

ch1993128 commented Feb 21, 2020

感谢,写得很详细,it matters。这真是一键解决所有问题了

@flin-X
Copy link

flin-X commented Feb 22, 2020

作为只要固件的小白,P3TERX/Actions-OpenWrt在固件的配置上确实要方便一些,openwrt软件包这么多,要配置出自己想要的,.config文件自己去写难度太大。
不得不说Microsoft是真的财大气粗,最近三个月同步过大佬代码的,不做修改,每天都会自动编译。

@coolsnowwolf coolsnowwolf unpinned this issue Feb 22, 2020
@Clackz
Copy link

Clackz commented Feb 22, 2020

感谢大佬的建议,也感谢P3TERX大佬写的脚本,不用自己改配置好多了……

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests