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

Build: Build Windows 7 binaries with a patched newer Go #4195

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

KobeArthurScofield
Copy link
Contributor

@KobeArthurScofield KobeArthurScofield commented Dec 22, 2024

该 pr 通过在 Actions 中先使用现有 Go 编译进行了改动后的 Go 后,使用改动过的 Go 编译出也许能在 Windows 7 上用的二进制。目前处于能构建并且构建出来的二进制的确能运行的阶段,但是手边没有安装了 win7 的机器或者镜像所以只能找小白鼠了。

目前使用 MetaCubeX/go 作为改版 Go 的来源,可以修改为其它仓库。

因为构建耗时较长,触发器相比一般的 build 少了对出现 go 文件发生更改时的触发项。

. ./all.bash

改为

. ./make.bash "$@" --no-banner
"$GOTOOLDIR/dist" banner

可以降低不少耗时,但是不保证编译出来的改版 Go 不会出奇怪的不易察觉的 break。

技术有限,目前编译出来的二进制不会显示 commit SHA。 更新:显示长 SHA

合并该 pr 需要同时合并 #4192。因为产生的打包文件名字一致。

@RPRX
Copy link
Member

RPRX commented Dec 22, 2024

合并该 pr 需要同时合并 #4192。因为产生的打包文件名字一致。

可以放同一个 pr 里吗

@KobeArthurScofield
Copy link
Contributor Author

可以放同一个 pr 里吗

完全可以,因为不知道反应会如何才拆开了

@KobeArthurScofield KobeArthurScofield marked this pull request as draft December 23, 2024 23:41
@KobeArthurScofield KobeArthurScofield marked this pull request as ready for review December 24, 2024 13:09
@KobeArthurScofield
Copy link
Contributor Author

更新:此 pr 现在采用 Meta 那边使用的方法,设置好 Go 之后将对应的补丁直接应用至工具链中再进行编译。

优点是速度快且不会有 dirty build 相关问题,编译后的界面外观和正常构建版本一致。

缺点是遇上类似 Go 1.21.5 那样的小变更需要用户反馈并且需要等待新补丁。(也可以自己维护一个分支)

应用此 pr 后可以关闭 #4192

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

Successfully merging this pull request may close these issues.

2 participants