gazebosim / gz-cmake

A set of CMake modules that are used by the C++-based Gazebo projects.
https://gazebosim.org/libs/cmake
Apache License 2.0
27 stars 31 forks source link

Remove check for s3cfg #308

Closed nkoenig closed 2 years ago

nkoenig commented 2 years ago

Signed-off-by: Nate Koenig nate@openrobotics.org

🦟 Bug fix

Summary

I moved the upload_doc.sh script from s3cfg to the aws CLI tool and forgot to remove the s3cfg check. This check is no longer needed, and prevents the nightly documentation job from completing successfully.

Checklist

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

j-rivero commented 2 years ago

I moved the upload_doc.sh script from s3cfg to the aws CLI tool and forgot to remove the s3cfg check. This check is no longer needed, and prevents the nightly documentation job from completing successfully.

Out of curiosity, how is the script authenticated to use the AWS cli client?

nkoenig commented 2 years ago

I moved the upload_doc.sh script from s3cfg to the aws CLI tool and forgot to remove the s3cfg check. This check is no longer needed, and prevents the nightly documentation job from completing successfully.

Out of curiosity, how is the script authenticated to use the AWS cli client?

The AWS cli client relies on the ~/.aws/credentials file, which stores the key information. The credentials file can be generated using:

aws configure set aws_access_key_id YOUR_ACCESS_KEY
aws configure set aws_secret_access_key YOUR_SECRET_KEY
aws configure set default.region us-east-1