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

Add PyQt6 to older Ubuntu versions with Pip #41946

Closed
wants to merge 3 commits into from

Conversation

InvincibleRMC
Copy link
Contributor

Purpose of using this:

To make Guis with PyQt6 on older Ubuntu Versions

Distro packaging links:

Links to Distribution Packages

https://pypi.org/project/PyQt6/

@InvincibleRMC InvincibleRMC requested a review from a team as a code owner July 3, 2024 01:05
@github-actions github-actions bot added the rosdep Issue/PR is for a rosdep key label Jul 3, 2024
@marcoag
Copy link
Contributor

marcoag commented Jul 10, 2024

I would say we usually don't accept pip packages that have a system version. In this case for focal and jammy it seems that pyqt5 is available, even though an older version there's still a system version of it. With that said let me still run it through the ROS Core meeting and come back with the outcome of that.

@marcoag marcoag self-assigned this Jul 10, 2024
@marcoag marcoag self-requested a review July 10, 2024 07:26
@marcoag
Copy link
Contributor

marcoag commented Jul 10, 2024

Adding a precedent: #41870 (review)

@marcoag marcoag requested a review from Yadunund July 10, 2024 07:43
@InvincibleRMC
Copy link
Contributor Author

Following up on this I would like to note that pyqt5 is available but eol https://endoflife.date/qt

@InvincibleRMC
Copy link
Contributor Author

If this solution does not get accepted would it be possible to add a python3-pyqt6-pip instead only for releases that don't have a pre-built package?

@marcoag
Copy link
Contributor

marcoag commented Jul 10, 2024

The problem is the releases you mentioned have a pre-built package, just that is from a previous version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rosdep Issue/PR is for a rosdep key
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants