-
Notifications
You must be signed in to change notification settings - Fork 2
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
Mailhog and ARM64 support #22
Comments
Adding to this conversation is this rather grim issue on Mailhog. And some similar projects have moved to mailpit |
Sad to hear that Mailhog is effectively on its way to hog heaven. Sounds like when we start making services with the Lando v4 API we should make one for mailpit... |
@almunnings out of curiosity is Mailhog just hella slow being emulated on ARM, or does it have fatal errors in running on ARM? Was messing with mssql today and had a fatal ARM emulation error, fixed it by using the "Use Rosetta for x86/amd64 emulation on Apple Silicon" in the "Features in Development" section of the Docker Desktop settings...don't know if that helps with performance speed. |
Nah the mailhog image works fine. It's 10mb, the speed isn't a concern. Just when creating the service, to avoid errors in the Lando start, we need to provide: overrides:
platform: linux/amd64 Eg mail:
type: mailhog
app_mount: false
portforward: true
overrides:
platform: linux/amd64
hogfrom:
- appserver So the issue is more about upstream support from the developer in the docker image space, as well as any innovations that all devs need... |
Issue isn't really an issue, more of a conversation starter:
Mailhog seems pretty stale (stable).
So stable it's missing out on arm support.
Theres no ARM64 support on docker hub.
A few other have been moving their images over to
anatomicjc/mailhog
(source) for ARM support.Would also like to suggest as a complete alternative:
Perhaps an idea is a selection of options for a generic mail plugin?
Email is an important part of development still. If part of "The Great Armification", I'd hate for a staple like this module to miss out.
Interim, a fix for arm64 is:
The text was updated successfully, but these errors were encountered: