-
-
Notifications
You must be signed in to change notification settings - Fork 487
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
In MPFR, don't delete old libraries #12548
Comments
This comment has been minimized.
This comment has been minimized.
Author: Jeroen Demeyer |
Diff for the mpfr spkg, for review only |
comment:2
Attachment: mpfr-2.4.2.p2.diff.gz |
This comment has been minimized.
This comment has been minimized.
comment:4
Looks good to me. Why even delete the headers? |
Reviewer: John Palmieri |
comment:5
Isn't the point of deleting the old librairies to not have tons of old librairies inside lib/ if Sage keeps on getting updated? You might argue that it's not a big deal, because Sage is already so big, but... And wasn't #12366 sufficient for building gcc? |
comment:7
Replying to @jpflori:
Probably. But this wouldn't work here, as deleting the old mpir/mpfr/mpc libraries would mean that gcc stops working. There are work-arounds, but they would make the build/upgrade process a lot more complicated.
For serial builds, #12366 is sufficient. The problem with parallel builds is that there is a cerain time between the deleting of the old mpfr libraries and the installation of the new mpfr libraries. During that time, a gcc in a parallel build wouldn't work anymore. |
Merged: sage-5.0.beta6 |
libmpfr (and libmpfr and libmpc, #12515) are run-time dependencies of
gcc
(#12369). These therefore become build-time dependencies of Sage. When these libraries are deleted during installation of mpfr, gcc stops working, causing build failures in parallel builds.Since there is no clear reason why the old libraries should be deleted, we simply don't delete them. The mpir
spkg-install
already doesn't delete libraries and mpc will be fixed in #12515.spkg: http://boxen.math.washington.edu/home/jdemeyer/spkg/mpfr-2.4.2.p2.spkg
CC: @jpflori
Component: packages: standard
Author: Jeroen Demeyer
Reviewer: John Palmieri
Merged: sage-5.0.beta6
Issue created by migration from https://trac.sagemath.org/ticket/12548
The text was updated successfully, but these errors were encountered: