Re: [PATCH] CI: install avocado debian packages from isar-build.org

[Anton Mikanovich <amikan@ilbers.de>]

22.10.2021 10:57, Henning Schild wrote:
> Might be better than doing it with pip. So we only need to maintain one
> path while waiting for upsteam debian to ship packages.
>
> Signed-off-by: Henning Schild <henning.schild@siemens.com>
> ---
>   scripts/ci_build.sh | 14 ++++++++------
>   1 file changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/scripts/ci_build.sh b/scripts/ci_build.sh
> index 5ff455e3d77b..ddec9e38cb87 100755
> --- a/scripts/ci_build.sh
> +++ b/scripts/ci_build.sh
> @@ -14,14 +14,16 @@ export PATH=$PATH:/sbin
>   # Go to Isar root
>   cd "$(dirname "$0")/.."
>   
> -# install avocado in virtualenv in case it is not there already
> +# install avocado in from deb.isar-build.org in case it is not there already
>   if ! command -v avocado > /dev/null; then
>       sudo apt-get update -qq
> -    sudo apt-get install -y virtualenv
> -    rm -rf /tmp/avocado_venv
> -    virtualenv --python python3 /tmp/avocado_venv
> -    source /tmp/avocado_venv/bin/activate
> -    pip install avocado-framework
> +    sudo apt-get install gnupg2 lsb-release curl -y
> +    codename=$( lsb_release -cs )
> +    curl http://deb.isar-build.org/debian-isar.key --output /tmp/isar-gpg.pub
> +    sudo apt-key add /tmp/isar-gpg.pub
> +    echo "deb http://deb.isar-build.org/debian-isar $codename-isar main" | sudo tee /etc/apt/sources.list.d/isar.list > /dev/null
> +    sudo apt-get update -qq
> +    sudo apt-get install -y avocado
>   fi
>   
>   # Get Avocado build tests path

What is the main difference of this patch and `[PATCH v10 17/17] 
ci_build: Install Avocado if needed` (besides distro detection and apt-get)?
Looks pretty much like the way we proposed originally.

-- 
Anton Mikanovich
Promwad Ltd.
External service provider of ilbers GmbH
Maria-Merian-Str. 8
85521 Ottobrunn, Germany
+49 (89) 122 67 24-0
Commercial register Munich, HRB 214197
General Manager: Baurzhan Ismagulov