Python3 support? changed the makefile to work with either python2/python3 #185

Merged
merged 1 commit into from Jan 11, 2015

Conversation

Projects
None yet
2 participants
@jeremyherbert
Contributor

jeremyherbert commented Jan 11, 2015

Hi,

First of all, cocotb is the best thing since sliced bread. :)

Anyway, it is listed in the README that Python2.6+ is supported. But it does not work for python3k. After modifying the makefile to make it build (pull request attached), it is clear that there have been some changes to the python C api which makes the current cocotb codebase incompatible. I used all of the latest 3.4.0 dev packages available on Ubuntu 14.04.1.

Is support for python3 planned? I am considering giving it a go myself, though I'm not sure my skills are up to the task.

@chiggs

This comment has been minimized.

Show comment
Hide comment
@chiggs

chiggs Jan 11, 2015

Collaborator

Hi Jeremy,

Thanks for this contribution and your supportive comments - it's always nice to hear that Cocotb is useful to other people.

Python 3 is something we would like to support, sadly to date it's simply something that has been buried in the eternally growing list of things to do. The same goes for ensuring pypy compatibility, which probably works but we may have introduced regressions.

We can set up a branch and enable Python3 and Pypy on travis and gradually fix up any issues. At least the amount of code that actually uses the C-API for Python is quite minimal.

Collaborator

chiggs commented Jan 11, 2015

Hi Jeremy,

Thanks for this contribution and your supportive comments - it's always nice to hear that Cocotb is useful to other people.

Python 3 is something we would like to support, sadly to date it's simply something that has been buried in the eternally growing list of things to do. The same goes for ensuring pypy compatibility, which probably works but we may have introduced regressions.

We can set up a branch and enable Python3 and Pypy on travis and gradually fix up any issues. At least the amount of code that actually uses the C-API for Python is quite minimal.

chiggs added a commit that referenced this pull request Jan 11, 2015

Merge pull request #185 from jeremyherbert/master
Python3 support? changed the makefile to work with either python2/python3

@chiggs chiggs merged commit d583b26 into potentialventures:master Jan 11, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment