You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by le...@apache.org on 2021/06/25 07:42:39 UTC
[tvm] branch main updated: [CI] Install curl in the context of
ubuntu_install_nodejs.sh (#8326)
This is an automated email from the ASF dual-hosted git repository.
leandron pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git
The following commit(s) were added to refs/heads/main by this push:
new aa56cc1 [CI] Install curl in the context of ubuntu_install_nodejs.sh (#8326)
aa56cc1 is described below
commit aa56cc13ecc28783eb5436128e8274a4f2e639f0
Author: Leandro Nunes <le...@arm.com>
AuthorDate: Fri Jun 25 08:42:13 2021 +0100
[CI] Install curl in the context of ubuntu_install_nodejs.sh (#8326)
* Make sure that curl is installed, as this script is used on
ci_lint, which does not need all the packages installed by
ubuntu_install_core.sh
---
docker/install/ubuntu_install_nodejs.sh | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/docker/install/ubuntu_install_nodejs.sh b/docker/install/ubuntu_install_nodejs.sh
index 79c2c3e..de3ba31 100755
--- a/docker/install/ubuntu_install_nodejs.sh
+++ b/docker/install/ubuntu_install_nodejs.sh
@@ -21,6 +21,10 @@ set -u
set -o pipefail
apt-get update
+# Please do not remove 'curl' package installation from here, as this
+# script runs in some images (e.g. ci_lint) that keep a very mininal
+# set of packages installed by default.
+apt-get install -y curl
# The node install script fetched and executed here will update the
# apt source list, hence the second apt-get update is necessary.