search

amazon-archives / aws-robomaker-sample-application-persondetection

Use AWS RoboMaker and demonstrate the use of Amazon Rekognition to recognize people's faces and Amazon Polly to synthesize speech.
MIT No Attribution
25 stars 18 forks source link

Depend on release-v2 branch of kinesisvideo-common #39

Closed juanrh closed 5 years ago

juanrh commented 5 years ago

Issue #, if available:

Description of changes: Depend on release-v2 branch of kinesisvideo-common, so we get the update on the checksum for the KVS SDK. Otherwise we get a build error as seem on https://travis-ci.org/aws-robotics/aws-robomaker-sample-application-persondetection/jobs/556938644 

Starting >>> kinesis_manager
--- stderr: kinesis_manager
CMake Warning at CMakeLists.txt:61 (message):
  Could not find GTest.  Not building unit tests.

CMake Error at /kinetic_ws/src/aws-robomaker-sample-application-persondetection/robot_ws/build/kinesis_manager/kvssdk/KVS_SDK_IMPORT-prefix/src/KVS_SDK_IMPORT-stamp/verify-KVS_SDK_IMPORT.cmake:27 (message):
  error: MD5 hash of

    /kinetic_ws/src/aws-robomaker-sample-application-persondetection/robot_ws/build/kinesis_manager/kvssdk/KVS_SDK_IMPORT-prefix/src/1.7.8.tar.gz

  does not match expected value

    expected: b8d1ac7ffb3626cb60c7e719edf84006
      actual: 9f3da8e64603198fe20d5c38e4a4cb08

make[2]: *** [kvssdk/KVS_SDK_IMPORT-prefix/src/KVS_SDK_IMPORT-stamp/KVS_SDK_IMPORT-download] Error 1
make[1]: *** [kvssdk/CMakeFiles/KVS_SDK_IMPORT.dir/all] Error 2

Note that travis job still passes, I'll also investigate that later

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

mm318 commented 5 years ago

I meant we should change this sample app to using HEAD of release branches for all the Cloud Extensions that it depends on.

I understand that would be a big change, so if you want to change only one Cloud Extension dependency for now, then just use the commit hash instead.