Skip to content

Conversation

@xqzr
Copy link
Contributor

@xqzr xqzr commented Mar 30, 2025

@RPRX
Copy link
Member

RPRX commented Mar 30, 2025

这次总该带上 darwin 了吧

我记得你加过好多 sockopt,不会都是 linux 的、没带上 win 和 mac 吧?要不你排查一下一个 PR 内全解决了,不要一个个 PR

@RPRX
Copy link
Member

RPRX commented Mar 30, 2025

还有有没有必要考虑一下 freebsd

@xqzr
Copy link
Contributor Author

xqzr commented Mar 30, 2025

不会都是 linux 的、没带上 win 和 mac 吧?要不你排查一下一个 PR 内全解决了,不要一个个 PR

是这样,因为以前只知道 Linux 写法。
记得在哪听过,一个 PR 只做一件事

@RPRX
Copy link
Member

RPRX commented Mar 30, 2025

一个 PR 只做一件事是对的,但有很多 typo 时总不能一个 typo 开一个 pr 吧,把三个主要系统的 sockopt 对齐、一个 PR 解决吧

freebsd 就不抢救了,因为我看到 Xray 的 releases 里还有 openbsd,算了,谁用谁写吧

@Fangliding
Copy link
Member

可以把 custom sockopt 开放到这几个系统

@RPRX RPRX merged commit d0350f8 into XTLS:main Mar 31, 2025
35 checks passed
@RPRX RPRX changed the title Sockopt: Add Windows V6Only Sockopt: Allow v6only work for Windows & Darwin Mar 31, 2025
@RPRX RPRX changed the title Sockopt: Allow v6only work for Windows & Darwin Sockopt: Allow listen v6only work for Windows & Darwin Mar 31, 2025
RPRX added a commit that referenced this pull request Mar 31, 2025
@RPRX
Copy link
Member

RPRX commented Mar 31, 2025

我把 Darwin 的代码加上了,应该能用,月底了想发个 3 月累积更新版本,先测测最近的修改有没有问题

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.

3 participants