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.
In section 2.5.3, shell variables:
> If a variable is initialized from the environment, it shall be marked for
export immediately; see the export special built-in.
That is the distinction of an environment variable inherited from the parent
process: it is marked for export "immediately", rather than when you use the
`export` special built-in.
--
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]
