Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 1 commit into from Jan 11, 2015
Merged

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

merged 1 commit into from Jan 11, 2015

Conversation

jeremyherbert
Copy link
Contributor

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
Copy link
Contributor

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
Python3 support? changed the makefile to work with either python2/python3
@chiggs chiggs merged commit d583b26 into cocotb:master Jan 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants