search

aws-samples / amazon-kinesis-video-streams-producer-embedded-c

Light-wight Amazon Kinesis Video Streams Producer SDK For FreeRTOS/Embedded Linux
Apache License 2.0
27 stars 17 forks source link

Simplify producer code structure #44

Closed weichihl closed 2 years ago

weichihl commented 2 years ago
  1. Remove shared and interface library. Keep static library only.
  2. Remove memory wrapper for dynamic linking.
  3. Clasify source code by folders.
  4. Move KvsApp into source folder.

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

codingspirit commented 2 years ago

Is there any peculiar reason that we are going to remove supports for shared build?

weichihl commented 2 years ago

I think the shared library makes the CMake project more complex. In the beginning, the shared libraries are for reducing memory usage if we want to execute WebRTC separately. Now there is a sample code that runs producer and WebRTC with static linking. So we expect we won't run producer and WebRTC separately. It will be easier to understand the CMake project structure if we remove shared libraries.