-
Notifications
You must be signed in to change notification settings - Fork 190
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
Avoid cgo dependencies #375
Comments
os/user dependency could be potentially removed by cs3org/reva#964 |
From https://golang.org/doc/go1.11
|
After further research it seems that we can't get rid of cgo fully as the glibc is requried for LDAP-DNS queries and mDNS lookups via golang stdlib. https://golang.org/pkg/net/#hdr-Name_Resolution Generally all of cgo can be disabled by CGO_ENABLED=0 but any problems with linking will surface during runtime. There are some very hacky workarounds which embed the dynamic lib in to the binary: |
todo: add tradeoffs from not using cgo |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 10 days if no further activity occurs. Thank you for your contributions. |
update: these are the current packages using CGo: ~/code/owncloud/ocis/ocis/cmd/ocis master
❯ go list -f "{{if .CgoFiles}}{{.ImportPath}}{{end}}" $(go list -f "{{.ImportPath}}{{range .Deps}} {{.}}{{end}}") ./...
github.com/mattn/go-sqlite3
net
os/user
plugin
runtime/cgo |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 10 days if no further activity occurs. Thank you for your contributions. |
@refs could you check again and close if applicable? |
following https://ops.tips/notes/glibc-golang-learnings/#how-can-i-see-what-requires-cgo
I see these cgo dependencies for ocis:
Ideas howto remove those?
The text was updated successfully, but these errors were encountered: