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 3 commits into
base: main
Choose a base branch
from

Conversation

KobeArthurScofield
Copy link
Contributor

该 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。

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

@RPRX
Copy link
Member

RPRX commented Dec 22, 2024

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

可以放同一个 pr 里吗

@KobeArthurScofield
Copy link
Contributor Author

可以放同一个 pr 里吗

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

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