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.

Sign up for GitHub

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

Jump to bottom

sage/misc/cython.py: make "def atlas()" deal with the Accelerate Framework on OSX #3563

Closed
sagetrac-mabshoff mannequin opened this issue Jul 6, 2008 · 4 comments
Closed

sage/misc/cython.py: make "def atlas()" deal with the Accelerate Framework on OSX #3563

sagetrac-mabshoff mannequin opened this issue Jul 6, 2008 · 4 comments
Assignees
@sagetrac-mabshoff
Labels
c: build p: critical / 2 t: bug
Milestone
sage-3.1.2

Comments

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jul 6, 2008

The review of #3530 exposed a bug hidden in sage/misc/cython.py: "def atlas()" would not return anything useful on OSX since we need to link against the Accelerate Framework.

Component: build

Issue created by migration from https://trac.sagemath.org/ticket/3563
@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-3.1.3 milestone Jul 6, 2008
@sagetrac-mabshoff sagetrac-mabshoff mannequin self-assigned this Jul 6, 2008
@sagetrac-mabshoff
Copy link
Mannequin Author

sagetrac-mabshoff mannequin commented Sep 14, 2008

comment:1

This issue actually breaks Cython on OSX 10.4 since the linker fails after it complains about a missing ATLAS.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-3.1.3, sage-3.1.2 Sep 14, 2008
@sagetrac-mabshoff
Copy link
Mannequin Author

sagetrac-mabshoff mannequin commented Sep 15, 2008

Attachment: trac_3563.2.patch.gz

@rlmill
Copy link
Mannequin

rlmill mannequin commented Sep 15, 2008

comment:3

Looks good to me, and

[05:11am] mabshoff: ok, fixed the tests, passes on OSX and not-OSX: #3563

@sagetrac-mabshoff
Copy link
Mannequin Author

sagetrac-mabshoff mannequin commented Sep 15, 2008

comment:4

Merged in Sage 3.1.2.rc4

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Sep 15, 2008
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sagetrac-mabshoff sagetrac-mabshoff

Labels
c: build p: critical / 2 t: bug
Projects
None yet
Milestone
sage-3.1.2
Development

No branches or pull requests
0 participants