Uploaded image for project: 'Qt for Python'
  1. Qt for Python
  2. PYSIDE-3215

binaries not exposed if installed with pdm with cache enabled

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Not Evaluated Not Evaluated
    • None
    • 6.10.0
    • PySide
    • None

      In my project I use pdm to manage my dependencies.
      among other things, pdm let me set up a central cache for all the installed packages

      Unfortunately it seems that the "binaries" (for example designer, linguist, etc..) offered by pyside packages do not work properly if installed in this way.

      If I try to run them all I get is a stacktrace that ends up with

      FileNotFoundError: [Errno 2] No such file or directory: '/home/zed/.cache/pdm/packages/pyside6_addons-6.10.0-cp39-abi3-manylinux_2_34_x86_64.whl.cache/PySide6/designer'

      ...witch is true, as the file is offered by pyside6_essentials, not pyside6_addons

      from a very quick view of the issue it seems that qt_tool_wrapper (in the module PySide6.scripts.pyside_tool) tries to get the executable absolute path from the PySIde6 "directory".

      def qt_tool_wrapper(qt_tool, args, libexec=False):
          # Taking care of pyside6-uic, pyside6-rcc, and pyside6-designer
          # listed as an entrypoint in setup.py
          pyside_dir = Path(ref_mod.__file__).resolve().parent
          if libexec and sys.platform != "win32":
              exe = pyside_dir / 'Qt' / 'libexec' / qt_tool
          else:
              exe = pyside_dir / qt_tool
      

      I think that, when trying to get the pyside_dir it's not correct (nor needed in the first place) to `.resolve()` the link.

      In fact in a simple test I did removing it the issue was fixed.

      Is it possible to apply this change?

      FYI: the related pdm bug is on https://github.com/pdm-project/pdm/issues/3651

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            crmaurei Cristian Maureira-Fredes
            zeed Vito De Tullio
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:

                There are no open Gerrit changes