xiaoxiang781216 commented on a change in pull request #5732:
URL: https://github.com/apache/incubator-nuttx/pull/5732#discussion_r825560041



##########
File path: tools/ci/cibuild.sh
##########
@@ -381,6 +381,26 @@ function rust {
   fi
 }
 
+function zig {
+  mkdir -p "${prebuilt}"/zig
+  add_path "${prebuilt}"/zig
+
+  if ! type zig &> /dev/null; then
+    case ${os} in
+      Darwin)
+        brew install zig
+        ;;
+      Linux)
+      # Currently Debian don't install zig (only built)
+        export ZIG_HOME=${prebuilt}/zig
+        cd /tmp
+       wget --quiet 
https://ziglang.org/download/0.9.1/zig-linux-x86_64-0.9.1.tar.xz

Review comment:
       change wget to curl?

##########
File path: tools/ci/cibuild.sh
##########
@@ -395,7 +395,7 @@ function zig {
         export ZIG_HOME=${prebuilt}/zig
         cd /tmp
        wget --quiet 
https://ziglang.org/download/0.9.1/zig-linux-x86_64-0.9.1.tar.xz
-       tar -xf zig-linux-x86_64-0.9.1.tar.xz && rm 
zig-linux-x86_64-0.9.1.tar.xz && mv zig-linux-x86_64-.0.9.1/* "${prebuilt}"/zig 
&& rm -fr zig-linux-x86_64-0.9.1

Review comment:
       merge to previous commit




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to