SG-33297 Fixups

[julien-lang]

This pull request refactors the tk_build_qt_resources module to improve the handling of compiler arguments and environment variables. The most important changes include switching from string-based command construction to list-based argument handling for subprocess calls, updating environment variable defaults, and refining paths for PySide2 tools.

Refactoring compiler argument handling:

• tk_toolchain/cmd_line_tools/tk_build_qt_resources/__init__.py: Modified build_qt to use compiler.extend() for adding arguments and switched subprocess calls to pass a list for better handling of command-line arguments.
• tk_toolchain/cmd_line_tools/tk_build_qt_resources/__init__.py: Updated build_ui and build_res functions to construct compiler as a list instead of a formatted string, ensuring safer and more flexible argument handling. [1] [2]

Environment variable updates:

• tk_toolchain/cmd_line_tools/tk_build_qt_resources/__init__.py: Changed the default value for the --pyenv argument to prioritize PYENV_VIRTUAL_ENV or VIRTUAL_ENV over PYTHONPATH, aligning with modern virtual environment practices.

Refinement of PySide2 tool paths:

• tk_toolchain/cmd_line_tools/tk_build_qt_resources/__init__.py: Updated the paths for pyside2-uic and pyside2-rcc to explicitly include the bin directory, ensuring compatibility with virtual environments.

[Copilot]

Pull Request Overview

This PR refactors the tk_build_qt_resources module to improve handling of compiler arguments and environment variables while refining tool paths for PySide2.

• Switches from string‐based to list-based argument handling for subprocess calls
• Updates the default environment variable to prioritize PYENV_VIRTUAL_ENV or VIRTUAL_ENV
• Modifies PySide2 tool paths to include the bin directory