Opened 13 years ago

Closed 13 years ago

#8445 closed defect (fixed)

sh: kpsewhich: not found - Sage 4.3.4.alpha0 on Solaris

Reported by: David Kirkby Owned by: John Palmieri
Priority: major Milestone: sage-4.3.4
Component: porting: Solaris Keywords:
Cc: Minh Van Nguyen, Jaap Spies, John Palmieri, Mitesh Patel, Sage Combinat CC user Merged in: sage-4.3.4.alpha1
Authors: John Palmieri Reviewers: Mike Hansen, David Kirkby
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges

Description (last modified by Minh Van Nguyen)

Background

After downloading the 4.3.4.alpha0.tar I applied some patches necessary to get this to build on Solaris 10 (SPARC). Namely:

Patches installed to allow Sage to build properly

  • #7867 A patch for Python which allows Python modules to be built properly.
  • #8440 A patch for Python to allow the _multiprocessing module to build.

Patches installed to allow most all doctests to pass in 4.3.3 (a few fail in 4.3.4.alpha0

  • #8374 Numerical noise in sage/symbolic/constants_c.pyx
  • #8375 Numerical noise in sage/symbolic/pynac.pyx
  • #8391 Change 'top' to 'prstat' on Solaris, othewise lots of doctests time out.
  • #8408 Update sqlite to the latest version (otherwise #8397, #8398, #8399, #8400 and #8401 all fail).

The problems

Running the long doctests I see:

sage -t  -long "devel/sage/sage/categories/finite_semigroups.py"
sh: kpsewhich: not found
sh: kpsewhich: not found
**********************************************************************
File "/export/home/drkirkby/32/sage-4.3.4.alpha0/devel/sage/sage/categories/finite_semigroups.py", line 232:
    sage: sorted(S.j_transversal_of_idempotents())
Expected:
    ['a', 'ab', 'ac', 'acb', 'b', 'bc', 'c']
Got:
    ['a', 'ab', 'ac', 'acb', 'b', 'c', 'cb']

So there are two problems.

  • kpsewhich: not found
  • doctest failure

But #8180 was supposed to fix this kpsewhich issue, so I believe the fix is not working fully.

I'll create a ticket for the test failure if needed. But I believe I see this mentioned on sage-devel, so it looks like I'm not the only one with this issue. The relevant ticket reporting this issue is #8430.

Attachments (1)

trac_8445-kpsewhich-solaris.patch (3.0 KB) - added by John Palmieri 13 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 13 years ago by Minh Van Nguyen

Cc: Sage Combinat CC user added

The sage-combinat team might be interested in this ticket.

comment:2 Changed 13 years ago by Minh Van Nguyen

Description: modified (diff)

comment:3 Changed 13 years ago by John Palmieri

Authors: John Palmieri
Status: newneeds_review

Here's a patch. It's hard for me to doctest it on Solaris: I think the only machine I have access to is t2.math, but there isn't enough room in /scratch for me to install Sage. So please test it out. It works for me on several other machines, and on t2, if I load a file containing the relevant code, that works as well.

The patch also reformats the warning messages that get printed if tkz-berge.sty (etc.) are not present.

Changed 13 years ago by John Palmieri

comment:4 Changed 13 years ago by David Kirkby

Thank's John,

I will test this, but it might take me a few days, as I have a very busy schedule this week.

Fortunately this is not a critical patch.

Dave

comment:5 Changed 13 years ago by David Kirkby

Owner: changed from David Kirkby to John Palmieri

John,

I've tested this on Solaris, and find no more "kpsewhich" problems. So from my point of view it is working.

However, I don't feel comfortable giving this a positive review, as I don't understand much of the code.

Perhaps one of the others cc'ed on the ticket can look over this, keeping in mind that it does solve the problem I reported.

Dave

comment:6 Changed 13 years ago by Mike Hansen

Status: needs_reviewpositive_review

This change looks good to me.

comment:7 Changed 13 years ago by Mike Hansen

Merged in: sage-4.3.4.alpha1
Resolution: fixed
Reviewers: Mike Hansen, David Kirkby
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.