Skip to content

Commit

Permalink
Unrolled build for rust-lang#127828
Browse files Browse the repository at this point in the history
Rollup merge of rust-lang#127828 - tgross35:docker-aarch64-uname, r=onur-ozkan

Commonize `uname -m` results for `aarch64` in docker runner

`uname -m` on Linux reports `aarch64`, but on MacOS reports `arm64`. Commonize this to `aarch64`.

With this fix, it is now possible to run aarch64 CI docker images on Arm MacOS.
  • Loading branch information
rust-timer committed Jul 18, 2024
2 parents e35364a + f3f0b57 commit c5b5547
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions src/ci/docker/run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,11 @@ do
shift
done

# MacOS reports "arm64" while Linux reports "aarch64". Commonize this.
machine="$(uname -m | sed 's/arm64/aarch64/')"

script_dir="`dirname $script`"
docker_dir="${script_dir}/host-$(uname -m)"
docker_dir="${script_dir}/host-${machine}"
ci_dir="`dirname $script_dir`"
src_dir="`dirname $ci_dir`"
root_dir="`dirname $src_dir`"
Expand Down Expand Up @@ -68,7 +71,7 @@ if [ -f "$docker_dir/$image/Dockerfile" ]; then

# Include the architecture in the hash key, since our Linux CI does not
# only run in x86_64 machines.
uname -m >> $hash_key
echo "$machine" >> $hash_key

# Include cache version. Can be used to manually bust the Docker cache.
echo "2" >> $hash_key
Expand Down Expand Up @@ -178,7 +181,7 @@ elif [ -f "$docker_dir/disabled/$image/Dockerfile" ]; then
build \
--rm \
-t rust-ci \
-f "host-$(uname -m)/$image/Dockerfile" \
-f "host-${machine}/$image/Dockerfile" \
-
else
echo Invalid image: $image
Expand All @@ -201,7 +204,7 @@ else
else
continue
fi
echo "Note: the current host architecture is $(uname -m)"
echo "Note: the current host architecture is $machine"
done

exit 1
Expand Down

0 comments on commit c5b5547

Please sign in to comment.