Skip to content

Commit

Permalink
Remove duplicate install command in CMakeLists.txt for api (#2474)
Browse files Browse the repository at this point in the history
  • Loading branch information
meastp authored Jan 4, 2024
1 parent 0134bbe commit 74db5e6
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions api/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,6 @@ if(OPENTELEMETRY_INSTALL)
LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
ARCHIVE DESTINATION ${CMAKE_INSTALL_LIBDIR})

install(
DIRECTORY include/opentelemetry
DESTINATION include
FILES_MATCHING
PATTERN "*.h"
PATTERN "metrics" EXCLUDE)

install(
DIRECTORY include/opentelemetry
DESTINATION include
Expand Down

1 comment on commit 74db5e6

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'OpenTelemetry-cpp sdk Benchmark'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 2.

Benchmark suite Current: 74db5e6 Previous: 0134bbe Ratio
BM_BaselineBuffer/2 10924915.307373453 ns/iter 2485297.3087168057 ns/iter 4.40
BM_BaselineBuffer/4 14222011.56616211 ns/iter 2602583.0245599514 ns/iter 5.46
BM_LockFreeBuffer/2 7446782.58895874 ns/iter 1053620.3384399414 ns/iter 7.07
BM_LockFreeBuffer/4 11038923.263549805 ns/iter 1162575.442215492 ns/iter 9.50

This comment was automatically generated by workflow using github-action-benchmark.

Please sign in to comment.