search

jd-boyd / python-lzo

Python bindings for the LZO data compression library
GNU General Public License v2.0
74 stars 41 forks source link

LZO_DIR vs CURL_DIR #2

Closed totaam closed 10 years ago

totaam commented 10 years ago

Clearly the setup.py never got tested on win32, and it must have been copied from the one for curl. Since it relies on knowing where the source is, I think it wouldn't hurt to add:

if not os.path.exists(LZO_DIR):
    raise Exception("please set LZO_DIR to where the lzo source lives")
jd-boyd commented 10 years ago

I have fixed the CURL_DIR references and added your suggested check. However, I still haven't tested this on win32. If you want to test it, that would be appreciated.

Windows support will likely remain untested by me until TravisCI supports windows.