#10233: Incomplete cython search path in setup.py
----------------------------+-----------------------------------------------
Reporter: vbraun | Owner: GeorgSWeber
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.6.2
Component: build | Keywords:
Author: Volker Braun | Upstream: N/A
Reviewer: | Merged:
Work_issues: |
----------------------------+-----------------------------------------------
Comment(by fbissey):
I am looking at this patch again. I am not sure about the last two lines
in this section:
{{{
extra_include_dirs = ['%s/include/python2.6'%SAGE_LOCAL,
# finally, standard C/C++ include dirs
'/usr/local/include/',
'/usr/include']
}}}
Are they really necessary? Shouldn't a standard compiler look into these
by default. After all the
other -I... arguments have been searched without success. Furthermore in a
scenario where
someone has installed their own compiler and want to use it rather the
system, and have even installed
there own libraries compiled with it (like a prefix basically but it
doesn't have to be that complete) that
means system headers may be put before their own installed headers. It is
not a very nice situation
which may be hard to debug.
Of course you could argue that it is not a likely scenario but it could
happen. There may be more
people than we think that use their own compilers rather than the system
one.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/10233#comment:10>
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 post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
