Skip to content

pyside6: use cmake to build the project and bring back cmake config files #11508

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

Merged
merged 1 commit into from
Jul 1, 2025

Conversation

rickliu2000
Copy link
Member

@rickliu2000 rickliu2000 commented Jun 27, 2025

Topic Description

  • pyside6: use cmake to build the project and bring back cmake config files
    Co-authored-by: Xinhui Yang [email protected]

Package(s) Affected

  • pyside6: 6.8.2.1-2

Security Update?

No

Build Order

#buildit pyside6

Test Build(s) Done

Primary Architectures

  • AMD64 amd64
  • AArch64 arm64
  • LoongArch 64-bit loongarch64

Secondary Architectures

  • Loongson 3 loongson3
  • PowerPC 64-bit (Little Endian) ppc64el
  • RISC-V 64-bit riscv64

@rickliu2000 rickliu2000 marked this pull request as ready for review June 27, 2025 05:19
@rickliu2000 rickliu2000 force-pushed the pyside6-cmake-config-fix branch from 28b24c5 to 67b178c Compare June 27, 2025 06:18
@aosc-buildit-bot
Copy link
Contributor

Dickens-topic report has been uploaded to pastebin as paste 3684906d-e5f2-423d-83b0-9b7f8d28c3f2.

@aosc-buildit-bot
Copy link
Contributor

Dickens-topic report has been uploaded to pastebin as paste 10185ff5-0dbe-449a-849f-99ea4af7ac9d.

@MingcongBai
Copy link
Member

@rickliu2000 @Cyanoxygen

This definitely isn't right:

+-rwxr-xr-x     ./usr/bin/android_deploy.py
+-rwxr-xr-x     ./usr/bin/deploy.py
+drwxr-xr-x     ./usr/bin/deploy_lib/
+-rwxr-xr-x     ./usr/bin/deploy_lib/__init__.py
+drwxr-xr-x     ./usr/bin/deploy_lib/android/
+-rwxr-xr-x     ./usr/bin/deploy_lib/android/__init__.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/android/android_config.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/android/android_helper.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/android/android_utilities.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/android/buildozer.py
+drwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/
+drwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/PySide6/
+-rwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/PySide6/__init__.tmpl.py
+drwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/shiboken6/
+-rwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/shiboken6/__init__.tmpl.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/commands.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/config.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/default.spec
+-rwxr-xr-x     ./usr/bin/deploy_lib/dependency_util.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/deploy_util.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/nuitka_helper.py
+-rwxr-xr-x     ./usr/bin/deploy_lib/pyside_icon.icns
+-rwxr-xr-x     ./usr/bin/deploy_lib/pyside_icon.ico
+-rwxr-xr-x     ./usr/bin/deploy_lib/pyside_icon.jpg
+-rwxr-xr-x     ./usr/bin/deploy_lib/python_helper.py
+-rwxr-xr-x     ./usr/bin/metaobjectdump.py
+-rwxr-xr-x     ./usr/bin/project.py
+drwxr-xr-x     ./usr/bin/project_lib/
+-rwxr-xr-x     ./usr/bin/project_lib/__init__.py
+-rwxr-xr-x     ./usr/bin/project_lib/design_studio_project.py
+-rwxr-xr-x     ./usr/bin/project_lib/newproject.py
+-rwxr-xr-x     ./usr/bin/project_lib/project_data.py
+-rwxr-xr-x     ./usr/bin/project_lib/utils.py

Copy link
Member

@MingcongBai MingcongBai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See my comment.

@Cyanoxygen
Copy link
Member

@rickliu2000 @Cyanoxygen

This definitely isn't right:

+-rwxr-xr-x     ./usr/bin/android_deploy.py

+-rwxr-xr-x     ./usr/bin/deploy.py

+drwxr-xr-x     ./usr/bin/deploy_lib/

+-rwxr-xr-x     ./usr/bin/deploy_lib/__init__.py

+drwxr-xr-x     ./usr/bin/deploy_lib/android/

+-rwxr-xr-x     ./usr/bin/deploy_lib/android/__init__.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/android/android_config.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/android/android_helper.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/android/android_utilities.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/android/buildozer.py

+drwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/

+drwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/PySide6/

+-rwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/PySide6/__init__.tmpl.py

+drwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/shiboken6/

+-rwxr-xr-x     ./usr/bin/deploy_lib/android/recipes/shiboken6/__init__.tmpl.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/commands.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/config.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/default.spec

+-rwxr-xr-x     ./usr/bin/deploy_lib/dependency_util.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/deploy_util.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/nuitka_helper.py

+-rwxr-xr-x     ./usr/bin/deploy_lib/pyside_icon.icns

+-rwxr-xr-x     ./usr/bin/deploy_lib/pyside_icon.ico

+-rwxr-xr-x     ./usr/bin/deploy_lib/pyside_icon.jpg

+-rwxr-xr-x     ./usr/bin/deploy_lib/python_helper.py

+-rwxr-xr-x     ./usr/bin/metaobjectdump.py

+-rwxr-xr-x     ./usr/bin/project.py

+drwxr-xr-x     ./usr/bin/project_lib/

+-rwxr-xr-x     ./usr/bin/project_lib/__init__.py

+-rwxr-xr-x     ./usr/bin/project_lib/design_studio_project.py

+-rwxr-xr-x     ./usr/bin/project_lib/newproject.py

+-rwxr-xr-x     ./usr/bin/project_lib/project_data.py

+-rwxr-xr-x     ./usr/bin/project_lib/utils.py

I've fixed that in the qt-6 branch. Let me pick that fix.

@Cyanoxygen Cyanoxygen force-pushed the pyside6-cmake-config-fix branch from 67b178c to fd10adc Compare June 30, 2025 09:03
@aosc-buildit-bot
Copy link
Contributor

Dickens-topic report has been uploaded to pastebin as paste b14a1b2b-ea87-4457-841b-57b7c851dc88.

@Cyanoxygen
Copy link
Member

…iles

* Install pyside6 scripts properly.
* Install pyside6 tools.

Co-authored-by: Xinhui Yang <[email protected]>
@Cyanoxygen Cyanoxygen force-pushed the pyside6-cmake-config-fix branch from fd10adc to 1645c64 Compare June 30, 2025 10:08
@aosc-buildit-bot
Copy link
Contributor

Dickens-topic report has been uploaded to pastebin as paste 97518a92-e7ab-463f-aa25-44df87721c82.

@MingcongBai MingcongBai dismissed Cyanoxygen’s stale review July 1, 2025 02:27

Addressed in the latest revision.

@Cyanoxygen Cyanoxygen merged commit 7151cd7 into stable Jul 1, 2025
6 checks passed
@Cyanoxygen Cyanoxygen deleted the pyside6-cmake-config-fix branch July 1, 2025 02:28
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.

4 participants