-
Notifications
You must be signed in to change notification settings - Fork 49
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
Python 3 #54
Comments
I was thinking someone ported it to support Python3, but really not, I could not find any pull request or issues, seems I have to find some time to work on it |
Thank you! |
It also depends on a bunch of long obsolete GNOME bindings which will also be going away because they don't support python3 such as gnome-python2* pygtk2-libglade, pyorbit, libbonobo* libgnome* and should be moved to GObject introspection based bindings |
Related Debian bug: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=933025 |
Please check if #58 solves your problem |
It would seem not: [gwyn@fedora64 ldtp]$ ldtp |
Please try again - I think you didn't try my python3 branch?! |
That branch seems to be based on 3.4.1; we're already on 3.5.0. |
I think my branch is up to date with master and git describe also seems to be fine
@limburgher why to you think so? |
ls ~/rpmbuild/BUILDROOT/ldtp-3.5.0-15.fc33.x86_64/usr/lib/python3.9/site-packages/ |
I'm not a Python expert but maybe there is some kind of misunderstanding. I took a look at the PR and it looks like the version string was changed from "3.5.1" to "3.4.1" in the setup.py maybe by mistake. |
Thanks @Werni2A for clarifying - I just wanted to highlight that the API might be incompatible. |
It's reporting 3.6.0 now. If I update my RPM spec to reflect that, and include the new ldtpme bits, I can build it. What's a fast test I can run on a progam I'm likely to have installed? :) |
I only use my xfce-test containers for ldtp... |
Python 2 is going away next year, are there plans to move to Python 3?
The text was updated successfully, but these errors were encountered: