Skip to content

Commit cb89b22

Browse files
authored
Make sure arkouda smoke testing uses CHPL_RE2=bundled (#29108)
Adjusts `smokeTest` to ensure we get a Chapel build with RE2 This logic used to exist, but was broke at some point. It then just happened to work because mason required RE2 too, but I recently removed mason smoke testing from the same config that tested arkouda, exposing this as an issue. [Reviewed by @benharsh]
2 parents ddffb27 + 199bef5 commit cb89b22

1 file changed

Lines changed: 12 additions & 5 deletions

File tree

util/buildRelease/smokeTest

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -151,11 +151,6 @@ if [ "$GITHUB_ACTIONS" != "true" ]; then
151151
source $CHPL_HOME/util/cron/common.bash
152152
fi
153153

154-
# Disable GMP and re2 to speed up build.
155-
if [ "$ARKOUDA_SMOKE_TEST" != "true" ]; then
156-
export CHPL_GMP=none
157-
fi
158-
159154
# mason currently requires CHPL_COMM=none -- https://github.com/chapel-lang/chapel/issues/12626
160155
COMM=`$CHPL_HOME/util/chplenv/chpl_comm.py`
161156
if [ "$COMM" != "none" ]; then
@@ -172,6 +167,18 @@ else
172167
export CHPL_RE2=none
173168
fi
174169

170+
171+
# Arkouda requires gmp and re2
172+
# if no arkouda required, disable gmp (re2 is already handled above)
173+
# otherwise, force enable gmp and re2 (even in a quickstart configuration)
174+
if [ "$ARKOUDA_SMOKE_TEST" != "true" ]; then
175+
export CHPL_GMP=none
176+
else
177+
export CHPL_GMP=bundled
178+
export CHPL_RE2=bundled
179+
fi
180+
181+
175182
echo ""
176183

177184
# Show me the environment!

0 commit comments

Comments
 (0)