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

fix compatibility issues with newer gvisor #3903

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

Conversation

hossinasaadi
Copy link
Contributor

No description provided.

@MHSanaei
Copy link
Contributor

OMG, finally! 😄

@Fangliding
Copy link
Member

Fangliding commented Oct 11, 2024

gomod停留在1.21是故意的 为了编译出和win7兼容的版本
所以要么把win7丢了要么把这个丢了要不又要分两个go文件了。。 参考这里 https://github.com/XTLS/Xray-core/blob/df370f0223ffdcb5e0408f18ff9bba866347c239/transport/internet/tls/ech_go121.go

@hossinasaadi
Copy link
Contributor Author

gomod停留在1.21是故意的 为了编译出和win7兼容的版本 所以要么把win7丢了要么把这个丢了要不又要分两个go文件了。。 参

@Fangliding thanks for details.
I think should be a way to separate win7 support :/

@RPRX
Copy link
Member

RPRX commented Oct 12, 2024

@RPRX
Copy link
Member

RPRX commented Oct 12, 2024

v25 给 win7 切换到那个需要改 go 源码或者额外 exe 的方案吧

@RPRX
Copy link
Member

RPRX commented Oct 15, 2024

不是,改了 go.mod 中的 go version,Win7 是怎么编译过的?记得当初测试结果是不能改

@RPRX
Copy link
Member

RPRX commented Oct 15, 2024

难道是因为当初 go version 没写到最小版本号

@Fangliding
Copy link
Member

action log好像显示了下载go1.23的日志

@RPRX
Copy link
Member

RPRX commented Oct 15, 2024

action log好像显示了下载go1.23的日志

这下抽象了,我试试把 go.mod 中的 go version 改回去

@RPRX
Copy link
Member

RPRX commented Oct 15, 2024

算了

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.

4 participants