cloudflare / tls-tris

crypto/tls, now with 100% more 1.3. THE API IS NOT STABLE AND DOCUMENTATION IS NOT GUARANTEED.
Other
291 stars 51 forks source link

Use standard import paths, preparing for Go modules #169

Closed Lekensteyn closed 5 years ago

Lekensteyn commented 5 years ago

Change import paths to use standard import paths, this is friendlier to standard tooling like syntax checking. Additionally, go mod vendor and go get now works without a custom Go compiler. Such import paths will also be needed for Go 1.13.

Note that go test passes except for example_test.go which fails due to importing crypto/tls instead of tris. Likewise for programs in _dev/.

When building with _dev/go.sh, it will still patch the import paths to use internal import paths though. This is the default and won't change.


Partially addresses #158 by making it importable by default.

Lekensteyn commented 5 years ago

This makes development using vim-go a nicer experience since autocompletion and lint now actually works (my main motivation for making this change now). In the future it might enable using Tris as a Go module, but that is a future task to verify (and might require changes to example_test.go).

kriskwiatkowski commented 5 years ago

LGTM