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

Change settings for INCLUDE_MPFR_PATCH from "1" or "0" to "yes" or "no" #7147

Closed
sagetrac-drkirkby mannequin opened this issue Oct 7, 2009 · 2 comments
Closed

Change settings for INCLUDE_MPFR_PATCH from "1" or "0" to "yes" or "no" #7147

sagetrac-drkirkby mannequin opened this issue Oct 7, 2009 · 2 comments

Comments

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Oct 7, 2009

Due to a bug in Solaris, MPFR can fail tests on sun4v machines.

We have solved this (#6453) some months back.,I added some code to a shell script that could apply the patch or not depending on whether INCLUDE_MPFR_PATCH was set to 1 (apply patch) or 0 (do not apply). Given other variable in Sage use "yes" or "no", it would be sensible this was changed to be consistent. Clearly this is a very trivial patch,

CC: david.kirkby@onetel.ne

Component: porting: Solaris

Issue created by migration from https://trac.sagemath.org/ticket/7147

@aghitza
Copy link

aghitza commented Oct 19, 2009

comment:2

Where's the patch?

@aghitza aghitza added this to the sage-4.2 milestone Oct 19, 2009
@sagetrac-drkirkby
Copy link
Mannequin Author

sagetrac-drkirkby mannequin commented Dec 3, 2009

comment:3

I'm closing this as invalid, as the variable can take on 3 values - 0, 1 and 2, not a yes/no.

@sagetrac-drkirkby sagetrac-drkirkby mannequin removed the s: needs info label Dec 3, 2009
@sagetrac-drkirkby sagetrac-drkirkby mannequin closed this as completed Dec 3, 2009
@mwhansen mwhansen removed this from the sage-4.3 milestone Dec 3, 2009
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants