-
Notifications
You must be signed in to change notification settings - Fork 30
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
Tests fail to convert #2
Comments
It works for me, tested with Python 3.2.3 and 3.3.5. What version of Python 3 are you using? Here's the generated file control.cc. |
Thanks for replying! Testing with 3.2.3 gave me this instead:
Seems to work better...(?) but not all the way. :( It's not apparent from the errors, but could I be missing some dependency? |
I'm not spending a lot of time trying to find what the problem is. I'm still on my vacation... I'll take some further dives into this when I'm back at work. :) However, I noticed that there definitely is some problem related to the python version:
So, apparently some unicode troubles... But with 3.4 it's (strangely) worse, and different:
But anyway, as mentioned, I'll look some more later. |
Thanks for investigating. It appears that Python 3.2 doesn't support the I've added support for Python 3.4's I was also able to reproduce the set of C++ errors you saw. It looks as if |
Oh, good to hear it's not only me! :) |
Cool, it's much smoother now! Conversion seems to be successful.
Btw, I see that in some messages, the unicode "u" is still there (e.g. strings with japanese)... |
I've added small descriptions to the "Discarding" messages in 8f90a7c |
Ah, I see... "not implemented" :) |
Some of the discarded tests are also Python specific, or test failures that are compile-time errors in C++. |
According to the README the (upstream) tests should pass.
Wanting to check the tests out (as a form of documentational examples), I tried the mentioned "make test".
This, however, fails with the current head (0d41413):
Should it work? I mean, is the trouble on my end?
I'm using Ubuntu 15.04.
The text was updated successfully, but these errors were encountered: