Skip to content

Commit

Permalink
Fix installer on ARM 64-bit
Browse files Browse the repository at this point in the history
## Why?

The QEMU multi-arch docker container should only be executed on x86 based
architectures.

## What?

Updated the installer to skip the QEMU docker container command if we are
not running on a x86 based architecture.
  • Loading branch information
sbkok committed Oct 24, 2024
1 parent 74329ab commit f6097d5
Showing 1 changed file with 8 additions and 3 deletions.
11 changes: 8 additions & 3 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
# SPDX-License-Identifier: Apache-2.0

# Makefile versions
MAKEFILE_VERSION := 2.3.1
MAKEFILE_VERSION := 2.3.2
UPDATE_VERSION := make/latest

# This Makefile requires Python version 3.9 or later
Expand Down Expand Up @@ -153,8 +153,13 @@ tox: deps
)

docker:
@echo "Prepare docker to support all architectures..."
docker run --rm --privileged multiarch/qemu-user-static --reset -p yes
( \
RUNNING_ON_ARCH=$$(arch); \
if [ "$$RUNNING_ON_ARCH" = "x86_64" ] || [ "$$RUNNING_ON_ARCH" = "i386" ]; then \
echo "Prepare docker to support the required architectures..." && \
docker run --rm --privileged multiarch/qemu-user-static --reset -p yes; \
fi \
)

version_number: .venv
@echo "Generate a new version number..."
Expand Down

0 comments on commit f6097d5

Please sign in to comment.