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

Docs: Add notes on 'build targets' on Get Started #1021

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

Conversation

josmperez
Copy link
Contributor

(Draft) Add a note about a problem running builds on Apple silicon to Get Started topic; add a second example to the Mage description.

Fixes #1012

@josmperez josmperez added the type/docs Changes only affect the documentation label Jul 22, 2024
Copy link

github-actions bot commented Jul 22, 2024

Hello! 👋 This repository uses Auto for releasing packages using PR labels.

✨ This PR can be merged. It will not be considered when calculating future versions of the npm packages and will not appear in the changelogs.

@josmperez josmperez marked this pull request as ready for review September 12, 2024 22:55
@josmperez josmperez requested a review from a team as a code owner September 12, 2024 22:55
@josmperez josmperez requested review from academo and removed request for a team September 12, 2024 22:55
@academo academo added the no-changelog Don't include in changelog and version calculations label Sep 13, 2024

List all available Mage targets for additional commands:

```bash
mage -l
```

:::note

There is a mismatch between AMD, ARM64, and the Docker container which may cause the backend Delve debugger dev env to fail to run on Apple silicon.
Copy link
Member

Choose a reason for hiding this comment

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

@oshirohugo can you confirm this is still the case?

Copy link
Contributor

Choose a reason for hiding this comment

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

This was improved, but it seems it still not perfect and it will only work in Apple silicon after two runs.
@jackw you worked on that, can you confirm?

Copy link
Collaborator

@jackw jackw Oct 15, 2024

Choose a reason for hiding this comment

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

@oshirohugo I'll do some testing here and get back to you. It was a while since I tried this debugger.

Either way you cannot have a mismatch between arch related things (go, dlv and docker image) as it prevents the debugger from connecting from within the container to the host.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Don't include in changelog and version calculations type/docs Changes only affect the documentation
Projects
Status: 🧑‍💻 In development
Development

Successfully merging this pull request may close these issues.

Fix mismatch in 'Build the backend' instructions
4 participants