#16148: Really enable cython caching
-------------------------------------------------+-------------------------
Reporter: jdemeyer | Owner:
Type: defect | Status: closed
Priority: major | Milestone: sage-6.2
Component: build | Resolution: fixed
Keywords: | Merged in:
Authors: Jeroen Demeyer | Reviewers: Volker
Report Upstream: N/A | Braun
Branch: | Work issues:
dfc4bf95f2aa6ee5e69d6754a594d3332a11e35f | Commit:
Dependencies: | Stopgaps:
-------------------------------------------------+-------------------------
Comment (by leif):
{{{
Cython.Compiler.Main.default_options['cache'] = True
}}}
is probably the wrong thing; it should be passed in `options` to
`cythonize()`.
But essentially `cythonize()` appears to be broken w.r.t. this, since it
does:
{{{
#!python
...
c_options = CompilationOptions(**options)
...
options = c_options
...
if hasattr(options, 'cache'):
if not os.path.exists(options.cache):
os.makedirs(options.cache)
}}}
and one may end up with
{{{
File "/scratch/sage/local/lib/python2.7/site-
packages/Cython/Build/Dependencies.py", line 739, in cythonize
if not os.path.exists(options.cache):
File "/scratch/sage/local/lib/python/genericpath.py", line 18, in exists
os.stat(path)
TypeError: coercing to Unicode: need string or buffer, bool found
}}}
as reported on sage-devel.
--
Ticket URL: <http://trac.sagemath.org/ticket/16148#comment:17>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
