Skip to content

Commit

Permalink
Fix unbound variable in activate script (conda-forge#1067)
Browse files Browse the repository at this point in the history
We were checking `CF_LIBARROW_ACTIVATE_LOGGING` without a default
value. This led to failure under `set -u`. For original report, see
<conda-forge#1065 (comment)>.
  • Loading branch information
maresb authored and h-vetinari committed Jun 2, 2023
1 parent 6c71407 commit e02d4df
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion recipe/activate.sh
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
# is benign and doesn't need to be deleted.

_la_log() {
if [ "$CF_LIBARROW_ACTIVATE_LOGGING" = "1" ]; then
if [ "${CF_LIBARROW_ACTIVATE_LOGGING:-}" = "1" ]; then
# The following loop is necessary to handle multi-line strings
# like for the output of `ls -al`.
printf '%s\n' "$*" | while IFS= read -r line
Expand Down

0 comments on commit e02d4df

Please sign in to comment.