amazon-kinesis-video-streams-producer-sdk-cpp icon indicating copy to clipboard operation
amazon-kinesis-video-streams-producer-sdk-cpp copied to clipboard

Test and accommodate Producer C CMake improvements

Open stefankiesz opened this issue 11 months ago • 1 comments

Issue #, if available:


What was changed?

  • The path to open-source libraries will not longer have /local/ in it
  • This PR is used to test the Producer C CMake changes PR for compatibility

Why was it changed?

  • Removing /local/ from path makes this build tree uniform with the Producer C build, so the same Producer C can find it's built libraries whether built alone or as a dependency.

How was it changed?

  • /local/ was removed from the relevant places

What testing was done for the changes?

  • Built locally on Intel Mac and M2 Mac
  • The CI test suite

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

stefankiesz avatar Mar 17 '24 22:03 stefankiesz

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 16.51%. Comparing base (eeb392f) to head (45c81b7).

:exclamation: Current head 45c81b7 differs from pull request most recent head 65e7d2c. Consider uploading reports for the commit 65e7d2c to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1154      +/-   ##
===========================================
+ Coverage    16.48%   16.51%   +0.03%     
===========================================
  Files           50       50              
  Lines         7019     7022       +3     
===========================================
+ Hits          1157     1160       +3     
  Misses        5862     5862              

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Mar 17 '24 23:03 codecov-commenter