-
Notifications
You must be signed in to change notification settings - Fork 11
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
mpreal.h has been removed #407
base: master
Are you sure you want to change the base?
Conversation
Hi @lobis, it would be better to include the library It would be an issue to add that package to the docker? |
It should not be a problem (I assume it's installed as However its been a while since I modify the docker image and perhaps something can break due to some other circumstances, so I wouldn't update the "main" tag (the one we use for CI) of the Docker image without testing. In fact we should transition from using the docker images I published in my profile to using an official I think from now on we should use https://github.com/rest-for-physics/rest-docker/issues to create requests / issues related to dependencies on the Docker image (I am assuming the official image will be hosted in this repository, after I fix the problems). |
Yes, it is installed this way. It has been added in the recent docker update? |
Yes it has been added, now we need to merge all PRs to update the image: |
mpreal.h has been removed because it caused conflicts with some versions of MPFR libraries.
This compatibility issue seemed to be originated by the
mpreal.h
header present in the framework.The
mpreal.h
is included inside the linux packagelibmpfrc++-dev