Opened 11 years ago

Closed 11 years ago

#9599 closed enhancement (fixed)

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

Reported by: ddrake Owned by: drkirkby
Priority: major Milestone: sage-4.6.2
Component: porting: Solaris Keywords:
Cc: wjp, SimonKing, malb, jhpalmieri 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 drkirkby)

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 malb 11 years ago.

Download all attachments as: .zip

Change History (15)

comment:1 Changed 11 years ago by wjp

  • Cc wjp added

comment:2 Changed 11 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 11 years ago by SimonKing

  • Cc SimonKing malb added

comment:4 Changed 11 years ago by drkirkby

  • Cc jhpalmieri added
  • Description modified (diff)
  • Summary changed from re-merge #1396 and insure that Sage starts on t2.math to re-merge #1396 and insure that Sage starts + passes doc tests on t2.math

comment:6 Changed 11 years ago by malb

  • Status changed from new to needs_review

comment:7 Changed 11 years ago by malb

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

comment:8 Changed 11 years ago by malb

  • Authors set to Simon King, John Perry, Martin Albrecht

comment:9 Changed 11 years ago by malb

  • Status changed from needs_review to positive_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 11 years ago by jdemeyer

  • Reviewers set to Martin Albrecht

Is there a reason the milestone is sage-5.0?

comment:11 Changed 11 years ago by malb

  • Milestone changed from sage-5.0 to sage-4.6.2

No :)

comment:12 Changed 11 years ago by jdemeyer

  • Work issues set to 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 11 years ago by malb

comment:13 Changed 11 years ago by malb

  • Work issues commit message deleted

done

comment:14 Changed 11 years ago by jdemeyer

  • Merged in set to sage-4.6.2.alpha3
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.