-
Notifications
You must be signed in to change notification settings - Fork 271
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
Docker broken build #726
Comments
Hmm, it's a bit strange that we install ihatemoney when running the container, as opposed to when building it... Also, by default, we install the release from Pypi which can be quite old (as it is now) @bnjbvr do you know if this is expected? Shouldn't we install ihatemoney when building the container? |
Hmm I can't recall exactly, but I guess the thinking was to try to get the latest release that was available, so that restarting/respawning the Docker instance would always get the more up to date version. That being said, if it's downloading from pypi and the pypi releases are old, this defeats this purpose. So yeah, it might be better to install ihatemoney when building the container, from the sources; with an automated build on the main branch, we can even ensure it'll always be up to date too! (And use a separate Docker instance for the build, so the build dependencies don't get bundled in the user-facing docker image.) |
It seems that Alpine changed something in the image… |
@ToadCast can you try adding Of course, we may need |
Hello, I'm trying to setup an instance with Docker, I cloned the repository on my machine, I followed the documentation with Docker. I changed few options in the
Dockerfile
:The
docker build
part worked just fine, but when I'm trying to run the container I face an error toBuilding wheel for greenlet
with following command:and the logs from it:
The text was updated successfully, but these errors were encountered: