Skip to content

Commit af9029e

Browse files
RuoqingHestefano-garzarella
authored andcommitted
build_container: Avoid pip install with &&
Even if we have `set -e`, if `pip install` fails, the script goes on because we use the && operator. Avoid this to make sure that we discover any errors with `pip install`. Signed-off-by: Ruoqing He <heruoqing@iscas.ac.cn>
1 parent bf93a63 commit af9029e

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

build_container.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,8 @@ fi
5151

5252
# Allow pip to install packages system wide
5353
rm /usr/lib/python3.*/EXTERNALLY-MANAGED
54-
pip3 install --no-cache-dir pytest pexpect boto3 pytest-timeout && apt purge -y python3-pip
54+
pip3 install --no-cache-dir pytest pexpect boto3 pytest-timeout
55+
apt purge -y python3-pip
5556

5657
# Install rustup and a fixed version of Rust.
5758
curl https://sh.rustup.rs -sSf | sh -s -- \

0 commit comments

Comments
 (0)