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
A portion of nostd::variant currently requires a Boost-licensed header.
This may be an issue for some commercial products.
Two alternatives to consider:
research if we can get that portion code contributed under Open Telemetry (Apache-style) license. That way we still satisfy the ABI stability guarantees.
I would propose we totally remove MPark variant - replacing it with a copy of implementation borrowed from absl::variant. I need this anyways to start supporting Visual Studio 2015 #314 ... Thus, removing MPark variant we will get rid of the Boost licensing issue.
A portion of nostd::variant currently requires a Boost-licensed header.
This may be an issue for some commercial products.
Two alternatives to consider:
research if we can get that portion code contributed under Open Telemetry (Apache-style) license. That way we still satisfy the ABI stability guarantees.
the other alternative is to compile with std Ability to use Standard Library instead of opentelemetry nostd classes #67 . That approach means that commercial products building from source with standard library cannot safely interop with other prebuilt Open Telemetry modules.
The text was updated successfully, but these errors were encountered: