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 mismatch in 'Build the backend' instructions #1012

Open
josmperez opened this issue Jul 15, 2024 · 1 comment · May be fixed by #1021
Open

Fix mismatch in 'Build the backend' instructions #1012

josmperez opened this issue Jul 15, 2024 · 1 comment · May be fixed by #1021
Assignees
Labels
type/docs Changes only affect the documentation

Comments

@josmperez
Copy link
Contributor

Comment from JackW:

There is a mismatch between amd and arm64 and the docker container which we should likely put a note about here. Previously we forced amd in the docker platform but that causes the backend delve dev env to fail to run on apple silicon.

Also not really sure if this should be added here but we have more than just build:[arch] as available targets. e.g.

| build:backend | build a production build for the current platform | mage -v build:backend |

Maybe we circle back to the whole backend build commands and expand them later on.

@josmperez josmperez added the type/docs Changes only affect the documentation label Jul 15, 2024
@josmperez josmperez self-assigned this Jul 15, 2024
@josmperez josmperez linked a pull request Jul 22, 2024 that will close this issue
@josmperez
Copy link
Contributor Author

@jackw - When you get a chance, could you review the linked comment above (from you) and let me know if you still think we need to say more about the mismatch between ar64 and docker container, and if so, draft a note on the topic?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/docs Changes only affect the documentation
Projects
Status: 💡 Ideation
Development

Successfully merging a pull request may close this issue.

1 participant