I'm not sure why realpath was used here; that does seem quite expensive.

On Wed, Oct 28, 2015 at 9:30 AM, Jeroen Demeyer <[email protected]> wrote:
> Hello,
>
> I am currently profiling Cython's dependency-checking code when compiling
> the SageMath Cython library. I am skipping the actual cythonization, so it's
> really just the first "for m in module_list" loop in cythonize().
>
> I noticed that about 10% of total time is spent in os.path.realpath() calls
> in Cython/Build/Dependencies.py. Do we really need realpath()? Isn't
> abspath() sufficient?
>
> These realpath() calls have been introduced in
>
>
> commit 1bfd57ef7b1a350135024b19a9352411a952c553
> Author: R. Andrew Ohana <[email protected]>
> Date:   Tue Apr 9 14:03:19 2013 -0700
>
>     copy sources and depends instead of headers
>
>
> Jeroen.
> _______________________________________________
> cython-devel mailing list
> [email protected]
> https://mail.python.org/mailman/listinfo/cython-devel
_______________________________________________
cython-devel mailing list
[email protected]
https://mail.python.org/mailman/listinfo/cython-devel

Reply via email to