-
-
Notifications
You must be signed in to change notification settings - Fork 487
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
Cannot import sage.libs.lcalc.lcalc_Lfunction #13351
Comments
comment:1
Update lcalc spkg, modifying lot of the Makefile (that we already modify and which is not that nice) available at This will also need a Sage library patch. |
Attachment: trac_13351.patch.gz Attachment: lcalc.diff.gz Spkg diff, for review only. |
This comment has been minimized.
This comment has been minimized.
Author: Jean-Pierre Flori |
This comment has been minimized.
This comment has been minimized.
comment:4
Does this fix #9165? |
comment:5
No idea :) give it a try. |
comment:7
I think its the last ticket needed to correctly build Sage and get a working version except for a few doctests. |
comment:8
I'm sorry, Jean-Pierre - I agree, but I simply have had no time at work to deal with my XP box (esp. since the 32-bit issue apparently is preventing proper rebasing, as discussed somewhere). I hope to try again over spring break in a couple weeks. |
comment:9
Replying to @jpflori:
I'm away at a conference, and don't have access to a WIndows machine, until Sunday. And then I'll have jetlag :-) |
comment:10
No hurry, it's just that its been rottijg around for some monthes, so Id like to get it ressurected :) |
comment:11
I still get import problems:
indeed:
note that
works. |
comment:13
Replying to @dimpase:
Did you install the patch to the Sage library and rebuilt it as well?
That's expected, this file is not touched and does not depend on the spkg. |
comment:14
yes, I certainly applied trac_13351.patch and ran sage -b. (on 5.7.beta2, but I guess it should not matter). |
comment:15
Replying to @dimpase:
what happened is that the Python extension didn't get rebuilt, as the patch didn't touch the corresponding pyx file! So I touched this file, and now everything works. Positive review. |
Reviewer: Dmitrii Pasechnik |
comment:17
It works on XP too! |
Merged: sage-5.9.beta0 |
Cygchecking the package shows that it looks for an inexistent libLfunction.so.
Indeed, this file is moved to libLfunction.dll on Cygwin to let Sage build.
But at runtime, it is the original .so file which is looked for (surely some DT_SONAME equivalent).
Letting libLfunction.so in place and symlinking it from libLfunction.dll seems more functional.
A better solution would to tell lcalc to build directly a .dll file on Cygwin (and .dylib on Mac OS X, etc.).
Updated spkg doing so at
http://perso.telecom-paristech.fr/~flori/sage/lcalc-1.23.p11.spkg
or
http://www.infres.enst.fr/~flori/sage/lcalc-1.23.p11.spkg
Apply attachment: trac_13351.patch
CC: @kcrisman @dimpase
Component: porting: Cygwin
Keywords: cygwin spkg lcalc
Author: Jean-Pierre Flori
Reviewer: Dmitrii Pasechnik
Merged: sage-5.9.beta0
Issue created by migration from https://trac.sagemath.org/ticket/13351
The text was updated successfully, but these errors were encountered: