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.