search

videlec / pypolymake

Python wrappers for polymake
https://pypi.python.org/pypi/pypolymake
3 stars 5 forks source link

setup.sh still depends on SAGE_LOCAL, SAGE_ROOT environment variables #2

Closed mkoeppe closed 8 years ago

videlec commented 8 years ago

Hi Matthias,

The setup.py script needs to locate the polymake headers and libraries... this is the only reason for SAGE_LOCAL and SAGE_ROOT.

mkoeppe commented 8 years ago

Hi Vincent, I'm new to writing Python packages and setup.sh, but I have the impression (e.g. from http://stackoverflow.com/questions/9849257/how-to-set-cflags-and-ldflags-to-compile-pycrypto) that it is expected that users simply provide environment variables such as CFLAGS, CXXFLAGS, LDFLAGS so that all libraries are found.

On Sun, Jul 24, 2016 at 1:22 AM, Vincent Delecroix <notifications@github.com

wrote:

Hi Matthias,

The setup.py script needs to locate the polymake headers and libraries... this is the only reason for SAGE_LOCAL and SAGE_ROOT.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/videlec/pypolymake/issues/2#issuecomment-234746538, or mute the thread https://github.com/notifications/unsubscribe-auth/AH9WhWsihLM1gVwB19uVscFIWAcNniHjks5qYqIjgaJpZM4JTdvr .

Dr. Matthias Koeppe . . . . . . . . . . . http://www.math.ucdavis.edu/~mkoeppe Professor, Mathematics Chair, Applied Mathematics (Graduate Group) . . http://appliedmath.ucdavis.edu

videlec commented 8 years ago

Sure... you might also need to set LD_LIBRARY_PATH before you launch python (if your install is not standard). But I am not sure how it behaves with the sage polymake package (I am testing it right now).