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

pyside6: ensure PySide6 can be imported and used #1808

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

considerate
Copy link
Contributor

Contribution checklist (recommended but not always applicable/required):

  • There's an automated test for this change
  • Commit messages or code include references to related issues or PRs (including third parties)
  • Commit messages are conventional - examples from the log include "feat: add changelog files to fixup hook", "fix(contourpy): allow wheel usage", and "test: add sqlalchemy2 test"

Fixes #1791

@considerate considerate changed the title Considerate/pyside6 test running pyside6: ensure PySide6 can be imported and used Sep 16, 2024
@considerate considerate force-pushed the considerate/pyside6-test-running branch from f98eb8d to 9163577 Compare September 16, 2024 14:37
Viktor Kronvall added 2 commits September 22, 2024 11:32
The `program` attribute of apps should be set to the path of an
executable, not to a package. This commit updates the `poetry` app
to point directly to the `poetry` executable.
The PySide6/__init__.py script is at runtime trying to find the
dependent Qt libraries in subdirectories of its path in the installed
site-packages directory. This fails because the dependent
pyside6-essentials and pyside6-addons packages aren't installed into the
same site-packages directory. As a work-around this commit symlinks
those files into the site-packages directory of the `pyside6` package.

This commit also adds a small application that imports
and verifies that PySide6 can be used after building.
@considerate considerate force-pushed the considerate/pyside6-test-running branch from 9163577 to 82bd472 Compare September 22, 2024 02:32
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.

PySide6 broken Overrides
1 participant