Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix executable compilation on macOS #237

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

nesk
Copy link
Contributor

@nesk nesk commented Mar 29, 2025

The docker run command is failing on macOS if the i386/debian image isn't already in cache, the following error is shown:

docker: no matching manifest for linux/arm64/v8 in the manifest list entries.

To fix this, we add a --platform=linux/amd64 flag to the command, as recommended by the following SO answer: https://stackoverflow.com/a/79440230/1513045

The `docker run` command is failing on macOS if the `i386/debian` image isn't already in cache, the following error is shown:

```
docker: no matching manifest for linux/arm64/v8 in the manifest list entries.
```

To fix this, we add a `--platform=linux/amd64` flag to the command, as recommended by the following SO answer: https://stackoverflow.com/a/79440230/1513045
@nesk nesk requested a review from GabrielaReyna as a code owner March 29, 2025 11:33
```

Explanation:

- `docker run --rm`: Runs a Docker container and removes it after execution.
- `--platform=linux/amd64`: Defines the platform to use inside the container.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This does not seem to be accurate. Inside the container the platform must be 32-bit x86, not 64-bit x86. The purpose of the option is to enable execution of x86 containers on ARM M-class processors.

@AtibQur Please restructure this suggestion to clarify that only MacOS users should add this option. It does not seem advisable to enable this across the board.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants