eli-schwartz commented on code in PR #45858:
URL: https://github.com/apache/arrow/pull/45858#discussion_r2021212646


##########
ci/scripts/cpp_build.sh:
##########
@@ -118,12 +118,36 @@ if [ "${ARROW_USE_MESON:-OFF}" = "ON" ]; then
     fi
   }
 
+  ORIGINAL_CC="${CC}"
+  if [ -n "${CC}" ]; then
+    if [ "${ARROW_USE_CCACHE}" = "ON" ]; then
+      CC="ccache ${CC}"
+    else
+      if command -v sccache; then
+        CC="sccache ${CC}"
+      fi
+    fi
+  fi
+
+  ORIGINAL_CXX="${CXX}"
+  if [ -n "${CXX}" ]; then
+    if [ "${ARROW_USE_CCACHE}" = "ON" ]; then
+      CXX="ccache ${CXX}"
+    else
+      if command -v sccache; then
+        CXX="sccache ${CXX}"
+      fi
+    fi
+  fi
   meson setup \

Review Comment:
   In section 2.12 of the Shell Command Language: 
https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html
   
   
   The shell execution environment is described as:
   
   > Variables with the export attribute, along with those explicitly exported 
for the duration of the command, shall be passed to the utility environment 
variables
   
   which works together with the docs you linked for the `export` special 
built-in command, that notes that the effect of `export XXX` is to mark the 
variable `XXX` with the export attribute.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to