Skip to content

Commit

Permalink
Fixed bug in ‘pharo::prepare_image’ where the image was always downlo…
Browse files Browse the repository at this point in the history
…aded again due to ‘is_file’ being used instead of ‘is_dir’ to test the existence of the ‘target’ directory.
  • Loading branch information
Rinzwind committed Aug 6, 2023
1 parent c488370 commit 2fd015a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pharo/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -270,8 +270,8 @@ pharo::prepare_image() {
local target="${SMALLTALK_CI_CACHE}/${smalltalk_name}"
local pharo_zeroconf="${target}/zeroconfig"

if ! is_file "${target}"; then
is_dir "${target}" || mkdir "${target}"
if ! is_dir "${target}"; then
mkdir "${target}"
pushd "${target}" > /dev/null
fold_start download_image "Downloading ${smalltalk_name} image..."
download_file "${pharo_image_url}" "${pharo_zeroconf}"
Expand Down

0 comments on commit 2fd015a

Please sign in to comment.