Re: [PATCH v6 15/15] ci_build: Install Avocado if needed

[Jan Kiszka <jan.kiszka@siemens.com>]

On 09.05.21 08:42, Anton Mikanovich wrote:
> To keep compatibility will the build systems without avocado-framework
> preinstalled add the code to install it from debian-isar repo.
> 
> Signed-off-by: Anton Mikanovich <amikan@ilbers.de>
> ---
>  scripts/ci_build.sh | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
> 
> diff --git a/scripts/ci_build.sh b/scripts/ci_build.sh
> index d20d588..58a4446 100755
> --- a/scripts/ci_build.sh
> +++ b/scripts/ci_build.sh
> @@ -28,6 +28,23 @@ for prog in ${DEPENDENCIES} ; do
>      fi
>  done
>  
> +# Install Avocado if needed
> +if [ ! -x "$(which avocado)" ] ; then
> +    set +e
> +    APT_LIST="/etc/apt/sources.list.d"
> +    ISAR_REPO="deb http://deb.isar-build.org/debian-isar
> +    ISAR_KEY="http://deb.isar-build.org/debian-isar.key

You want to validate this key against a known-good checksum, even more
as you are downloading via http. Or embed the key into the repo.

> +    sudo -s <<EOSUDO
> +    mkdir -p $APT_LIST
> +    echo "$ISAR_REPO" > $APT_LIST/isar.list
> +    apt-key adv --fetch-keys $ISAR_KEY
> +    apt update
> +    yes | apt install avocado
> +EOSUDO
> +    echo "Installed Avocado-framework"
> +    set -e
> +fi
> +
>  show_help() {
>      echo "This script builds the default Isar images."
>      echo
> 

Jan

-- 
Siemens AG, T RDA IOT
Corporate Competence Center Embedded Linux