-
-
Notifications
You must be signed in to change notification settings - Fork 69
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
(GSoC) Port Cricket to use Toga, instead of Tkinter #65
Conversation
Hi there! In trying to test this end to end on local machine, I ran into an error. This is probably me doing something wrong, but I'd like to double-check so I can help review. My steps:
This results in the following error:
I know that this PR depends on the changes interface from beeware/toga#201, which makes me think I'm installing something wrong? If @Dayof or @freakboy3742 have thoughts, I'd love to help review this. |
@phildini AFAIK, It should just be a matter of using the #201 PR branch; if that's not working, I'll have to dig deeper. I haven't had a chance to take this code for a run myself, yet. |
@phildini Looking at your details: If you've done a This is required because the base toga package is just there to define dependencies; and pip goes to PyPI for those dependencies by default. |
I blew away my venv and started over, and still got an error... do I need to setup more packages than src/core, src/cocoa, toga, and cricket? https://gist.github.com/phildini/a46d507a32e9de9de9d818696fdad1ad |
@phildini I cleaned my env and I installed all over again and didn't find these errors, I'll open an vm to test this. |
@Dayof if both you and @freakboy3742 are able to get this running on your machines, then don't worry about trying to remote-debug my machine. @freakboy3742 is probably more qualified to review anyways. :) |
Detail about this PR on issue #58 !
This PR is dependent of the PR beeware/toga#201 .
Demo : https://youtu.be/5kz_CmQYFYE.