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

OS X binaries should issue a better warning on incompatible CPUs #6414

Closed
sagetrac-GeorgSWeber mannequin opened this issue Jun 25, 2009 · 11 comments
Closed

OS X binaries should issue a better warning on incompatible CPUs #6414

sagetrac-GeorgSWeber mannequin opened this issue Jun 25, 2009 · 11 comments

Comments

@sagetrac-GeorgSWeber
Copy link
Mannequin

sagetrac-GeorgSWeber mannequin commented Jun 25, 2009

Currently, every now and then a user reports on sage-support, that he got an error message like

/Applications/sage-4.0.1-OSX10.5-PowerPC-PowerMacintosh-Darwin/sage/
local/bin/sage-sage: line 198:   407 Illegal instruction     sage-
ipython "$@" -i

This is e.g. the case if a Sage binary built on a MacPPC with a G5 processor (typically the one the OS X 10.5 bdist is created on) is used on a MacPPC with only a G4 processor.

For the *nix world on Intel/AMD processors, the sage-flags.txt file was created for just the purpose to check whether the CPU is sufficient, to let a certain Sage binary run.

We seem to need something in that direction for OS X, too (though momentarily only for the PPC processors, not the Intel ones).

CC: @dimpase

Component: distribution

Reviewer: Jeroen Demeyer

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

@kcrisman
Copy link
Member

comment:1

Is this still an issue with the new checks for moving OSX etc?

@kcrisman
Copy link
Member

kcrisman commented Jan 5, 2015

comment:2

This is essentially the same as #5950, except newer and with a little more info, so I'm keeping it. Both of these should be addressed, I guess. From #5950:

-bdist should add something analog to the SSE flags on Linux so that if someone tried to run a ppc build on an x86 it aborts with a sane error message instead of just blowing up.

@jdemeyer
Copy link

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link

comment:3

There should be no warning needed, the binaries should be produced to run on any CPU.

@kcrisman
Copy link
Member

comment:5

There should be no warning needed, the binaries should be produced to run on any CPU.

? That doesn't really work on OS X all the time, we have seen many error messages on ask.sagemath for this for different instruction sets.

@kcrisman
Copy link
Member

comment:7

(By which I mean to say this isn't just a PPC/Intel issue.)

@dimpase
Copy link
Member

dimpase commented Apr 11, 2016

comment:8

it does not work all the time because of problems in the building process (forgetting to set proper flags), or perhaps toolchain bugs, or both, I don't really know.

SAGE_FAT_BINARY cooked with wrong FAT... :-)

@jdemeyer
Copy link

comment:9

Replying to @dimpase:

it does not work all the time because of problems in the building process (forgetting to set proper flags), or perhaps toolchain bugs, or both, I don't really know.

I haven't heard of any such bug recently. In any case, such things should be fixed on a case-by-case basis, I don't think that there is a general fix possible. So I still think that this ticket should be closed.

@dimpase
Copy link
Member

dimpase commented Apr 11, 2016

comment:10

Replying to @jdemeyer:

Replying to @dimpase:

it does not work all the time because of problems in the building process (forgetting to set proper flags), or perhaps toolchain bugs, or both, I don't really know.

I haven't heard of any such bug recently.

certainly with 6.9 binaries it is the case (e.g. reported today on sage-support). There were also very entertaining bugs like this reported by someone with a Mac box having a non-standard (upgraded?) CPU...
Don't recall about 7.1/2 right now.

By the way: faq-usage says: Nobody has yet figured out how to build Sage in such a way that MPIR and ATLAS work on all hardware.

In any case, such things should be fixed on a case-by-case basis, I don't think that there is a general fix possible. So I still think that this ticket should be closed.

@jdemeyer
Copy link

comment:11

OK, then let's close this ticket not as "worksforme" but as "invalid" because it's too general.

@kcrisman
Copy link
Member

comment:12

That seems like a good compromise.

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

4 participants