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

New mpfr-2.4.1.p1.spkg works with Open Solaris 64 bit #8068

Closed
jaapspies opened this issue Jan 25, 2010 · 6 comments
Closed

New mpfr-2.4.1.p1.spkg works with Open Solaris 64 bit #8068

jaapspies opened this issue Jan 25, 2010 · 6 comments

Comments

@jaapspies
Copy link
Member

Let spkg-install handle SAGE64="yes" on Open Solaris 64 bit.

See here:
http://boxen.math.washington.edu/home/jsp/ports/mpfr-2.4.1.p1.spkg

PASS: tpow_all
====================
All 148 tests passed
====================
make[2]: Leaving directory `/export/home/jaap/Downloads/sage-4.3.1/spkg/build/mpfr-2.4.1.p1/src/tests'
make[1]: Leaving directory `/export/home/jaap/Downloads/sage-4.3.1/spkg/build/mpfr-2.4.1.p1/src/tests'
make[1]: Entering directory `/export/home/jaap/Downloads/sage-4.3.1/spkg/build/mpfr-2.4.1.p1/src'
make[1]: Nothing to be done for `check-am'.
make[1]: Leaving directory `/export/home/jaap/Downloads/sage-4.3.1/spkg/build/mpfr-2.4.1.p1/src'

real 2m15.371s
user 1m22.294s
sys 0m54.607s
Successfully installed mpfr-2.4.1.p1

Jaap

CC: @sagetrac-drkirkby @williamstein

Component: porting

Author: Jaap Spies

Reviewer: David Kirkby

Merged: sage-4.3.3.alpha0

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

@jaapspies

This comment has been minimized.

@jaapspies
Copy link
Member Author

Attachment: mpfr-2.4.1.p1.patch.gz

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Jan 27, 2010

comment:3

This works fine. In fact, the library was already building 64-bit for me, without this patch, but I believe it is safer to have this. I suspect mpfr might actually ignore the CFLAGS and work out what it needs itself, as it is uses mpir, and that would already be configured 64-bit. But I would agree this is desirable to add this.

Positive review.

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Jan 27, 2010

Reviewer: David Kirkby

@qed777
Copy link
Mannequin

qed777 mannequin commented Feb 11, 2010

Merged: sage-4.3.3.alpha0

@qed777 qed777 mannequin removed the s: positive review label Feb 11, 2010
@qed777 qed777 mannequin closed this as completed Feb 11, 2010
@fchapoton

This comment has been minimized.

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