You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Cl 395882 added a GC-friendly runtime-backed concurrent map to the boring package which is being used in several crypto packages.
We (Microsoft) are patching Go to add support for OpenSSL as crypto backend at microsoft/go (github.com), and it would be convenient for us, and also for other companies which forked Go for the same purpose, to be able to use crypto/internal/boring.Cache without importing the whole boring package.
It worth nothing that we could also implement boring.Cache in our fork, but that would also mean patching runtime/mgc.go, which I would like to avoid in order to reduce conflict likeliness.
Would it be possible to move crypto/internal/boring/cache.go to a new package, i.e., crypto/internal/boring/cache.
Cl 395882 added a GC-friendly runtime-backed concurrent map to the boring package which is being used in several
crypto
packages.We (Microsoft) are patching Go to add support for OpenSSL as crypto backend at microsoft/go (github.com), and it would be convenient for us, and also for other companies which forked Go for the same purpose, to be able to use
crypto/internal/boring.Cache
without importing the wholeboring
package.It worth nothing that we could also implement
boring.Cache
in our fork, but that would also mean patchingruntime/mgc.go
, which I would like to avoid in order to reduce conflict likeliness.Would it be possible to move
crypto/internal/boring/cache.go
to a new package, i.e.,crypto/internal/boring/cache
.@rsc @FiloSottile
The text was updated successfully, but these errors were encountered: