Menu

#39 api_exists() in setup.py is broken

open
nobody
None
5
2012-02-13
2012-02-13
Eric
No

The regex is bad. It basically catches everything. Here's the diff for that line to a version that works correctly:

- if re.search(r'CS_PUBLIC|CS_RETCODE %s' % func, text):
+ if re.search(r'(CS_PUBLIC|CS_RETCODE) %s' % func, text):

Thanks!

Discussion


Log in to post a comment.