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

add persistent SAGE64 building switch on OSX and Solaris #22

Closed
williamstein opened this issue Sep 12, 2006 · 9 comments
Closed

add persistent SAGE64 building switch on OSX and Solaris #22

williamstein opened this issue Sep 12, 2006 · 9 comments

Comments

@williamstein
Copy link
Contributor

Should there be something to keep people from building on 32-bit then 64-bit then 32-bit.

Component: build

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

@williamstein
Copy link
Contributor Author

comment:1
[01:10] <william> the problem was doing something like "sage -br" on both a 32-bit and 64-bit machine but
[01:10] <william> with a shared filesystem.
[01:10] <mabshoff> Ah, that makes sense.
[01:10] <william> I.e., when you build sage for one architecture, maybe you shouldn't be allowed to
[01:10] <william> do "sage -br" or "sage -i" if the architectrue doesn't match.
[01:11] <mabshoff> okay,
[01:11] <william> or something like that.

@sagetrac-mabshoff sagetrac-mabshoff mannequin added this to the sage-2.10 milestone Sep 11, 2007
@williamstein
Copy link
Contributor Author

comment:3
[10:01] <wstein> regarding #22 the idea is that people might do "sage -br" on one machine, then login to another machine that
[10:02] <wstein> nsf mounts the same directory, and muck things up.
[10:02] <wstein> We should cache "uname -p" in SAGE_ROOT, and if it changes not allow "sage -br" unless the user
[10:02] <wstein> (or sage -i) or sage -upgrade, unless the user manually deletes the file.
[10:03] <wstein> Thoughts?

@sagetrac-mabshoff sagetrac-mabshoff mannequin self-assigned this Nov 26, 2007
@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-2.10, sage-2.8.15 Nov 26, 2007
@garyfurnish garyfurnish mannequin assigned garyfurnish and unassigned sagetrac-mabshoff Mar 21, 2008
@garyfurnish
Copy link
Mannequin

garyfurnish mannequin commented Mar 23, 2008

comment:7

Fixed in #1261 (PBuild)

@garyfurnish garyfurnish mannequin added c: build and removed c: basic arithmetic labels Apr 4, 2008
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jan 16, 2009

Attachment: trac_22.patch.gz

@sagetrac-mabshoff sagetrac-mabshoff mannequin changed the title 32/64/32-bit building switch add persistent SAGE64 building switch on OSX and Solaris Jan 16, 2009
@sagetrac-mabshoff sagetrac-mabshoff mannequin modified the milestones: sage-3.4.1, sage-3.3 Jan 16, 2009
@sagetrac-mabshoff sagetrac-mabshoff mannequin assigned sagetrac-mabshoff and unassigned garyfurnish Jan 16, 2009
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jan 16, 2009

comment:11

Mhh, looking at the complete ticket history this patch does not cover all what needs to be fixed on this ticket. I will post a patch on top that also fixes this issue.

Cheers,

Michael

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jan 16, 2009

comment:12

Unfortunately the trick William suggested does not work everywhere, i.e. even on the latest stable Debian release this happens:

mabshoff@modular:~$ uname -p
unknown

From man uname on that box:

       -p, --processor
              print the processor type or "unknown"

So we might want to make that part of the ticket a followup ticket.

Cheers,

Michael

@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jan 16, 2009

comment:13

Having thought about this and played around a little with uname it seems that it will not work and is not fine grained enough anyway. I would suggest to do write a small C program that identifies the following:

  • mode: i.e. 32, 64 bit
  • os: linux, osx, solaris, freebsd, cygwin
  • release: this would be the distribution on Linux, OSX 10.4/10.5, Solaris 10/Solaris 11/Opensolaris and so on

The way we can properly identify the build platform and decide more intelligently if we issue a warning, i.e running the Fedora 10 build on a Fedora 9 box should abort since it doesn't work. The test should be wrapped in a shell script since the binary will obviously only run on a subset of arches, i.e. if the binary fails to run we just about and print a canned warning together with a config info saved as text that is created when building the binary.

This is enough a task to split it off to a new ticket. I have some basic code that does some of the above already for OSX since I need this kind of code while cleaning up the build system.

Thoughts?

Cheers,

Michael

@rlmill
Copy link
Mannequin

rlmill mannequin commented Jan 22, 2009

comment:14

This is a "perfect patch."

... ;-)

@rlmill rlmill mannequin added the s: positive review label Jan 22, 2009
@sagetrac-mabshoff
Copy link
Mannequin

sagetrac-mabshoff mannequin commented Jan 23, 2009

comment:15

Merged in Sage 3.3.alpha1.

Note that the remaining issue from this ticket has been moved to #5062.

Cheers,

Michael

@sagetrac-mabshoff sagetrac-mabshoff mannequin closed this as completed Jan 23, 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

1 participant