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.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.