Avoid qt-main clobbering

traversaro commented 1 month ago

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

conda-forge-webservices[bot] commented 1 month ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

traversaro commented 1 month ago

Existing checks seems to be working fine thanks to https://github.com/conda-forge/pyside2-feedstock/pull/222, @hmaarrfk do you hae any additional test that you think it could make sense to add?

traversaro commented 1 month ago

@conda-forge-admin, please rerender

hmaarrfk commented 1 month ago

i know that it removes more just uic can you list out the binaries that we miss?

traversaro commented 1 month ago

i know that it removes more just uic can you list out the binaries that we miss?

The relevant code is https://github.com/pyside/pyside-setup/blob/dev/sources/pyside-tools/CMakeLists.txt#L35-L60 . All the tools that are skipped by the NO_QT_TOOLS actually come from the qt6 installation prefix, the precise list is:

On Apple platform, the directories:

while on non-Apple platform, also the following commands:

hmaarrfk commented 1 month ago

Alright thanks. Let’s go with this and see where it goes

conda-forge / pyside2-feedstock