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

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

Closed
sagetrac-drkirkby mannequin opened this issue Mar 5, 2010 · 9 comments
Closed

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

sagetrac-drkirkby mannequin opened this issue Mar 5, 2010 · 9 comments

Comments

@sagetrac-drkirkby
Copy link
Mannequin

sagetrac-drkirkby mannequin commented Mar 5, 2010

== 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 ===

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

== 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.

CC: @sagetrac-mvngu @jaapspies @jhpalmieri @qed777 @sagetrac-sage-combinat

Component: porting: Solaris

Author: John Palmieri

Reviewer: Mike Hansen, David Kirkby

Merged: sage-4.3.4.alpha1

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

@sagetrac-drkirkby sagetrac-drkirkby mannequin added this to the sage-4.3.4 milestone Mar 5, 2010
@sagetrac-drkirkby sagetrac-drkirkby mannequin self-assigned this Mar 5, 2010
@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Mar 5, 2010

comment:1

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

@sagetrac-mvngu

This comment has been minimized.

@jhpalmieri
Copy link
Member

Author: John Palmieri

@jhpalmieri
Copy link
Member

comment:3

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.

@sagetrac-drkirkby
Copy link
Mannequin Author

sagetrac-drkirkby mannequin commented Mar 6, 2010

comment:4

Attachment: trac_8445-kpsewhich-solaris.patch.gz

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

@sagetrac-drkirkby
Copy link
Mannequin Author

sagetrac-drkirkby mannequin commented Mar 6, 2010

comment:5

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

@sagetrac-drkirkby sagetrac-drkirkby mannequin assigned jhpalmieri and unassigned sagetrac-drkirkby Mar 6, 2010
@mwhansen
Copy link
Contributor

mwhansen commented Mar 6, 2010

comment:6

This change looks good to me.

@mwhansen
Copy link
Contributor

mwhansen commented Mar 7, 2010

Reviewer: Mike Hansen, David Kirkby

@mwhansen
Copy link
Contributor

mwhansen commented Mar 7, 2010

Merged: sage-4.3.4.alpha1

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

2 participants