Opened 12 years ago

Closed 12 years ago

#9599 closed enhancement (fixed)

re-merge #1396 and insure that Sage starts + passes doc tests on t2.math

Reported by: Dan Drake Owned by: David Kirkby
Priority: major Milestone: sage-4.6.2
Component: porting: Solaris Keywords:
Cc: wjp, Simon King, Martin Albrecht, John Palmieri Merged in: sage-4.6.2.alpha3
Authors: Simon King, John Perry, Martin Albrecht Reviewers: Martin Albrecht
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by David Kirkby)

A patch at #1396 was merged in 4.5.2.alpha0, but that caused Sage to segfault when starting on t2.math. That ticket was backed out in 4.5.2.alpha1, and should be re-merged. See #9583 for discussion.

In order to be of help for anyone looking at this ticket in some months or years time, the following description of t2.math may be useful.

Hardware and software configuration of t2.math.washington.edu

Attachments (1)

trac_9599.patch (45.8 KB) - added by Martin Albrecht 12 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 Changed 12 years ago by wjp

Cc: wjp added

comment:2 Changed 12 years ago by wjp

To summarize the lengthy discussion on #9583:

#1396 exposed a symbol clash between Singular and Pari with some linkers.

To fix this, we rename Singular's mu to Kstd1_mu. This could be done either with the SPKG at #9583 which changes only this, or with the SPKG at #8059 which upgrades Singular to a new version that includes the renamed mu.

comment:3 Changed 12 years ago by Simon King

Cc: Simon King Martin Albrecht added

comment:4 Changed 12 years ago by David Kirkby

Cc: John Palmieri added
Description: modified (diff)
Summary: re-merge #1396 and insure that Sage starts on t2.mathre-merge #1396 and insure that Sage starts + passes doc tests on t2.math

comment:6 Changed 12 years ago by Martin Albrecht

Status: newneeds_review

comment:7 Changed 12 years ago by Martin Albrecht

The rebased patch passes doctests for me. It still needs testing on t2.

comment:8 Changed 12 years ago by Martin Albrecht

Authors: Simon King, John Perry, Martin Albrecht

comment:9 Changed 12 years ago by Martin Albrecht

Status: needs_reviewpositive_review

The patch applies fine to 4.6.1.rc1 and doctests pass on t2, except:

sage -t  -long -force_lib devel/sage/sage/tests/cmdline.py # 19 doctests failed, timeouts
sage -t  -long -force_lib devel/sage/sage/parallel/decorate.py # 1 doctests failed, OSError: [Errno 12] Not enough space
sage -t  -long -force_lib devel/sage/sage/misc/trace.py # 2 doctests failed, 

These don't seem related to this patch.

The patch did have a positive review before and the only issue was a SIGSEGV on startup on t2. I'll thus set this patch to a positive review.

comment:10 Changed 12 years ago by Jeroen Demeyer

Reviewers: Martin Albrecht

Is there a reason the milestone is sage-5.0?

comment:11 Changed 12 years ago by Martin Albrecht

Milestone: sage-5.0sage-4.6.2

No :)

comment:12 Changed 12 years ago by Jeroen Demeyer

Work issues: commit message

The commit message will have to be changed to reflect number, also you should split it over multiple lines if it's that long (but make sure the first line makes sense by itself).

Changed 12 years ago by Martin Albrecht

Attachment: trac_9599.patch added

comment:13 Changed 12 years ago by Martin Albrecht

Work issues: commit message

done

comment:14 Changed 12 years ago by Jeroen Demeyer

Merged in: sage-4.6.2.alpha3
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.