Bazaar setup.py doesn't handle dev version of pyrex
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Bazaar |
New
|
Undecided
|
Unassigned |
Bug Description
I had trouble installing Bazaar from source. After some digging, I traced the problem to how setup.py handles the dev version of pyrex. When pyrex dev versions have been installed, pyrex_version in the code below has a '-dev' suffix. 'int' doesn't handle this well and throws an uncaught exception.
Here's the change I made to the setup.py file in order to get it to work.
Sorry about the clumsiness. I didn't checkout a repo and couldn't generate a proper patch easily.
Line 205
Old version:
pyrex_
This is the output of pyrex_version.
['0', '19-dev']
My version
import re
get_version_info = lambda x: int( re.sub("[^0-9]", "", x ) )
pyrex_
pyrex_
My weak version of the fix for this issue was insufficient given the craziness of pyrex and cython versioning, needs revisiting and making more robust.