-
-
Notifications
You must be signed in to change notification settings - Fork 791
Ensure app name is normalised, and comes from an explicit source #3931
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
Conversation
mhsmith
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep discussion about the substance of the change in one place in #3926.
I don't know what's happening with the GTK testbed, but the Qt failures look like they were caused by the most recent changes.
|
The GTK failure was transient; the Qt issue was caused because I forgot the testbed explicitly sets the app name (using the underscore variant) because of the code sharing between the two testbeds. |
Makes 2 small changes to the determination of app name:
This fixes #3926, which identified that running
python -m toga_demoresulted in the wrong app icon (because the app icon was evaluated asresources/toga_demo, whereas if you ran withbriefcase, you would get the icon (because Briefcase would embed the icon into the app binary)It also fixes the fact that
togawasn't listed as a dependency (because of the use of setuptools-dynamic-dependencies), and that some of the other definitions needed for successful Briefcase operation weren't listed.The app won't run on iOS, Android or Web, so those configurations are now disabled.
PR Checklist: