Fix exporting of Jaeger exporter target name

[seemk]

I believe it was introduced in #918

When doing make install the generated opentelemetry-cpp-config.cmake was trying to find a non-existent target, thus the OPENTELEMETRY_CPP_LIBRARIES variable did not have Jaeger exporter set.

[codecov]

Codecov Report

Merging #925 (4f6a8ad) into main (1f31341) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #925   +/-   ##
=======================================
  Coverage   95.44%   95.44%           
=======================================
  Files         158      158           
  Lines        6749     6749           
=======================================
  Hits         6441     6441           
  Misses        308      308           

[lalitb]

Sorry, my bad. Thanks for raising the PR for the fix.
Just wondering if we should instead revert the EXPORT_NAME to jaeger_trace_exporer to be consistent with how other targets are listed in OPENTELEMETRY_CPP_LIBRARIES variable.

opentelemetry-cpp/exporters/jaeger/CMakeLists.txt

Line 21 in 1f31341

PROPERTIES EXPORT_NAME opentelemetry_exporter_jaeger_trace)

[seemk]

Sorry, my bad. Thanks for raising the PR for the fix.
Just wondering if we should instead revert the EXPORT_NAME to jaeger_trace_exporer to be consistent with how other targets are listed in OPENTELEMETRY_CPP_LIBRARIES variable.

opentelemetry-cpp/exporters/jaeger/CMakeLists.txt

Line 21 in 1f31341

PROPERTIES EXPORT_NAME opentelemetry_exporter_jaeger_trace)

Thanks! Changed the export name instead 👍