Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-27776

ActiveQt does not install enough headers

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: P1: Critical
    • Resolution: Done
    • Affects Version/s: 5.0.0 Beta 2
    • Fix Version/s: 5.0.0
    • Component/s: ActiveX Support
    • Labels:
      None
    • Commits:
      2fe214ca90cfda68126bf6e83e876a3a00635521,7d4051d3bbc341574ca7a9bdf31d5f14306c1367

      Description

      The public header qaxselect.h includes ui_qaxselect.h. I don't know if other include problems exist. I noticed this one when testing if #include <ActiveQt> works. (Update: this part is now fixed)

      The Qt CI system likely did not catch this as the install of Qt is not autotested at all. Tests are run against the build-dir. (See also QTBUG-27315)

      Also, the fact that headers are in the 'ActiveQt' directory instead of a directory prefixed with 'Qt' should be considered a bug that can be fixed for Qt 5.0. The porting script from QtGui/QWidget to QtWidgets/QWidget can probably be extended to help there.

      qtactiveqt also does not follow other Qt conventions, as it does not use the qt_module.prf, and it represents multiple libraries instead of one. It should really be two modules - AxContainer and AxServer.

        Attachments

          Issue Links

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

            Activity

              People

              Assignee:
              buddenha Oswald Buddenhagen
              Reporter:
              stephen.kelly Stephen Kelly (Unused account) (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              13 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Gerrit Reviews

                  There are no open Gerrit changes