search

aws-samples / service-catalog-engine-for-terraform-os

Apache License 2.0
130 stars 40 forks source link

Add jq to Pre-requisites in README #16

Closed brakf closed 1 year ago

brakf commented 1 year ago

Hello,

While deploying the project, I noticed that the jq command-line JSON processor is required but not mentioned in the pre-requisites section of the README. The absence of jq can lead to errors during deployment.

To help others avoid running into this issue, I suggest updating the README to include jq in the pre-requisites section.

Thank you!

smaly-amazon commented 1 year ago

Hello,

Thank you for pointing this out. We will update the Readme to include this.

kshn01 commented 1 year ago

Hello,

We also need to install python3.9-distutils. I faced issue while running the script./bin/bash/deploy-tre.sh -r.

Thank you.

smaly-amazon commented 1 year ago

@kshn01

Thanks for pointing that out.

We just merged a change that uses a virtual environment for the automated installation and manual installation instructions. Python setuptools is included when creating a virtual environment.

We use setuptools which is a superset of distuitls. So I think the change to use a virtual environment will take care of this.