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

libm4ri not respecting SAGE_FAT_BINARY #9381

Closed
sagetrac-mariah mannequin opened this issue Jun 29, 2010 · 6 comments
Closed

libm4ri not respecting SAGE_FAT_BINARY #9381

sagetrac-mariah mannequin opened this issue Jun 29, 2010 · 6 comments

Comments

@sagetrac-mariah
Copy link
Mannequin

sagetrac-mariah mannequin commented Jun 29, 2010

libm4ri is not respecting SAGE_FAT_BINARY
on x86-Linux, specifically sse3 if available
will be used.

A mercurial patch is attached which when
SAGE_FAT_BINARY="yes", comments out the
configure test for sse3 - so configure reports
that sse3 does not exist on the machine.

Upstream: Reported upstream. Developers acknowledge bug.

Component: packages: standard

Author: Mariah Lenox

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

@sagetrac-mariah sagetrac-mariah mannequin added this to the sage-4.5 milestone Jun 29, 2010
@sagetrac-mariah
Copy link
Mannequin Author

sagetrac-mariah mannequin commented Jun 29, 2010

comment:1

Attachment: libm4ri-FAT-remove-sse3-usage.patch.gz

@williamstein
Copy link
Contributor

comment:2
  • It appears your patch modified the configure script that is auto-generated by autoconf?

  • It would be best to somehow change the m4ri configure.ac (?) so that you can tell it to not use sse. This would be good to get upstreamed into m4ri. I've emailed the developers of m4ri about this ticket, and hope they can address it.

@williamstein
Copy link
Contributor

Upstream: Reported upstream. Little or no feedback.

@malb
Copy link
Member

malb commented Jul 6, 2010

comment:4

Hi, upstream here

I think we should add an option to M4RI to not check for SSE instead. I can provide an updated M4RI later this week or next week the latest.

@malb
Copy link
Member

malb commented Jul 6, 2010

Changed upstream from Reported upstream. Little or no feedback. to Reported upstream. Developers acknowledge bug.

@malb
Copy link
Member

malb commented Jul 11, 2010

comment:5

cf.#9475 (thus I'd say we can close this ticket as a duplicate)

@rlmill rlmill mannequin added r: duplicate and removed p: major / 3 labels Jul 11, 2010
@rlmill rlmill mannequin removed this from the sage-4.5 milestone Jul 11, 2010
@rlmill rlmill mannequin removed the s: needs review label Jul 11, 2010
@rlmill rlmill mannequin closed this as completed Jul 11, 2010
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