cybozu-go / transocks

Transparent SOCKS5 / HTTP proxy in Go
MIT License
467 stars 61 forks source link

Use golang.org/x/sys instead of syscall? #13

Closed otariidae closed 5 years ago

otariidae commented 5 years ago

While installing transocks on i686 system, I got the following error:

go/src/github.com/cybozu-go/transocks/original_dst_linux.go:14:3: undefined: syscall.SYS_GETSOCKOPT

The official document says:

Deprecated: this package is locked down. Callers should use the corresponding package in the golang.org/x/sys repository instead. That is also where updates required by new systems or versions should be applied. See https://golang.org/s/go1.4-syscall for more information.

I manually replaced syscall with golang.org/x/sys/unix. It worked well.

For better OS support, use golang.org/x/sys instead?

ymmt2005 commented 5 years ago

Thank you for the report!

We should replace it as you described. Are you willing to make a pull request for it?

otariidae commented 5 years ago

Thank you for the quick reply. I submitted #14. Please take a look.