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

pyside6-deploy silently fills in qml_files and excluded_qml_plugins parameters in pysidedeploy.spec if they are intentionally empty

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Not Evaluated
    • 6.7
    • 6.6.2
    • Tooling
    • None
    • Linux/Wayland

    Description

      pyside6-deploy sets qml_files and excluded_qml_plugins parameters in the qt section of the pysidedeploy.spec file unless some are specified. Unless I use QML, I'd expect to have qml_files empty, but the tool disallows that. The same goes for excluded_qml_plugins.

      I suggest checking that the parameters are present and empty, and then leave them as they are. If the script fills them back in, it should at least warn about that.

      To reproduce the issue,

      • make a new directory,
      • make a new venv there, activate it,
      • pip install PySide6-Essentials,
      • place an empty main.py there,
      • run pyside6-deploy,
      • clear the parameters in pysidedeploy.spec:
        [qt]
        # comma separated path to qml files required
        # normally all the qml files required by the project are added automatically
        qml_files =
        # excluded qml plugin binaries
        excluded_qml_plugins =
        
      • re-run pyside6-deploy.

      The expected behavior is that the pysidedeploy.spec file remains intact.

      The actual behavior is that the parameters are back to their initial values.


      Surprisingly, on Windows, the pysidedeploy.spec file doesn't change, but the components are listed as command line parameters, making it too long to run. As I don't have a Windows PC, I can't provide more info on that.

      Attachments

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

        Activity

          People

            shpremna Shyamnath Premnadh
            stsav012 Anton Yablokov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes