The committers listed above are authorized under a signed CLA.

• ✅ login: step-security-bot / name: StepSecurity Bot (1d8da44)

✅ Deploy Preview for opentelemetry-cpp-api-docs canceled.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.69%. Comparing base (7801cd9) to head (ad21e5c).

@@            Coverage Diff             @@
##             main    #3384      +/-   ##
==========================================
+ Coverage   89.67%   89.69%   +0.02%     
==========================================
  Files         211      211              
  Lines        6832     6832              
==========================================
+ Hits         6126     6127       +1     
+ Misses        706      705       -1     

#3355 should already fix this problem.
Could you please try it?
__is_trivially_assignable is not available in some old toolchains which are maintained by community.

#3355 should already fix this problem. Could you please try it? __is_trivially_assignable is not available in some old toolchains which are maintained by community.

Thanks, @owent. Looks like the suggested patch fixes the issue so I'll close this PR.

@owent I spoke too soon. Looks like we are still seeing the same issue even with the suggested patch:

bazel-out/k8-opt/bin/external/io_opentelemetry_cpp/api/_virtual_includes/api/opentelemetry/nostd/type_traits.h:145:73: error: builtin __has_trivial_copy is deprecated; use __is_trivially_copyable instead [-Werror,-Wdeprecated-builtins]
  145 |   static constexpr bool value = std::is_copy_constructible<T>::value && __has_trivial_copy(T);
      |                                                                         ^
bazel-out/k8-opt/bin/external/io_opentelemetry_cpp/api/_virtual_includes/api/opentelemetry/nostd/type_traits.h:157:70: error: builtin __has_trivial_assign is deprecated; use __is_trivially_assignable instead [-Werror,-Wdeprecated-builtins]
  157 |   static constexpr bool value = std::is_copy_assignable<T>::value && __has_trivial_assign(T);
      |                                                                      ^
2 errors generated.

@owent I spoke too soon. Looks like we are still seeing the same issue even with the suggested patch:

bazel-out/k8-opt/bin/external/io_opentelemetry_cpp/api/_virtual_includes/api/opentelemetry/nostd/type_traits.h:145:73: error: builtin __has_trivial_copy is deprecated; use __is_trivially_copyable instead [-Werror,-Wdeprecated-builtins]
  145 |   static constexpr bool value = std::is_copy_constructible<T>::value && __has_trivial_copy(T);
      |                                                                         ^
bazel-out/k8-opt/bin/external/io_opentelemetry_cpp/api/_virtual_includes/api/opentelemetry/nostd/type_traits.h:157:70: error: builtin __has_trivial_assign is deprecated; use __is_trivially_assignable instead [-Werror,-Wdeprecated-builtins]
  157 |   static constexpr bool value = std::is_copy_assignable<T>::value && __has_trivial_assign(T);
      |                                                                      ^
2 errors generated.

Sorry, there were some mistakes before, could you please try the latest version of #3355 ?

@owent I spoke too soon. Looks like we are still seeing the same issue even with the suggested patch:

bazel-out/k8-opt/bin/external/io_opentelemetry_cpp/api/_virtual_includes/api/opentelemetry/nostd/type_traits.h:145:73: error: builtin __has_trivial_copy is deprecated; use __is_trivially_copyable instead [-Werror,-Wdeprecated-builtins]
  145 |   static constexpr bool value = std::is_copy_constructible<T>::value && __has_trivial_copy(T);
      |                                                                         ^
bazel-out/k8-opt/bin/external/io_opentelemetry_cpp/api/_virtual_includes/api/opentelemetry/nostd/type_traits.h:157:70: error: builtin __has_trivial_assign is deprecated; use __is_trivially_assignable instead [-Werror,-Wdeprecated-builtins]
  157 |   static constexpr bool value = std::is_copy_assignable<T>::value && __has_trivial_assign(T);
      |                                                                      ^
2 errors generated.

Sorry, there were some mistakes before, could you please try the latest version of #3355 ?

Yes, that worked. I'll close this one out. Thanks!